[android] Fixed Mopub ad's shows tracking

2 files changed, 10 insertions, 2 deletions

diff --git a/android/src/com/mapswithme/maps/ads/CompoundNativeAdLoader.java b/android/src/com/mapswithme/maps/ads/CompoundNativeAdLoader.java
index 8517f2fd76..7c89b6c0d9 100644
--- a/android/src/com/mapswithme/maps/ads/CompoundNativeAdLoader.java
+++ b/android/src/com/mapswithme/maps/ads/CompoundNativeAdLoader.java
@@ -73,7 +73,15 @@ public class CompoundNativeAdLoader extends BaseNativeAdLoader implements Native
       mLoaders.add(loader);
       attach();
       loader.setAdListener(this);
-      loader.loadAd(context, banner.getId());
+      // TODO: this workaround is need to avoid memory leak of activity context in MyTarget SDK.
+      // The fix of myTarged sdk will be done in this issue https://jira.mail.ru/browse/MOBADS-207.
+      // After the mentioned issued is fixed, this workaround should be removed. Also, we can't use
+      // the application context for all providers, because some of them (e.g. Mopub) requires an
+      // activity context and can't work with application context correctly.
+      if (loader instanceof MyTargetAdsLoader)
+        loader.loadAd(context.getApplicationContext(), banner.getId());
+      else
+        loader.loadAd(context, banner.getId());
     }
   }
 
diff --git a/android/src/com/mapswithme/maps/ads/MopubNativeDownloader.java b/android/src/com/mapswithme/maps/ads/MopubNativeDownloader.java
index b34aeefdb5..0d7105e2b9 100644
--- a/android/src/com/mapswithme/maps/ads/MopubNativeDownloader.java
+++ b/android/src/com/mapswithme/maps/ads/MopubNativeDownloader.java
@@ -95,7 +95,7 @@ class MopubNativeDownloader extends CachingNativeAdLoader
   @Override
   public void onAdImpressed()
   {
-    // No op.
+    LOGGER.d(TAG, "on MoPub Ad impressed");
   }
 
   @Override