(android) added bannerHeight to banner LOAD event (#361)

src/android/banner/BannerExecutor.java

@@ -24,6 +24,10 @@ public class BannerExecutor extends AbstractExecutor {
      * The adView to display to the user.
      */
     private AdView adView;
+
+    public AdView getAdView() {
+        return adView;
+    }
     /**
      * if want banner view overlap webview, we will need this layout
      */
@@ -35,8 +39,11 @@ public class BannerExecutor extends AbstractExecutor {
 
     boolean bannerVisible = false;
 
+    public AdMob plugin;
+
     public BannerExecutor(AdMob plugin) {
         super(plugin);
+        this.plugin = plugin;
     }
 
     @Override

src/android/banner/BannerListener.java

@@ -1,8 +1,11 @@
 package name.ratson.cordova.admob.banner;
 
+import android.util.DisplayMetrics;
 import android.util.Log;
+import android.view.ViewTreeObserver;
 
 import com.google.android.gms.ads.AdListener;
+import com.google.android.gms.ads.AdView;
 
 import org.json.JSONException;
 import org.json.JSONObject;
@@ -43,14 +46,39 @@ public void onAdLeftApplication() {
         executor.fireAdEvent("onLeaveToAd", data);
     }
 
+    private int pxToDp(int px) {
+        DisplayMetrics displayMetrics = executor.plugin.cordova.getActivity().getApplicationContext().getResources().getDisplayMetrics();
+        int dp = Math.round(px / (displayMetrics.xdpi / DisplayMetrics.DENSITY_DEFAULT));
+        return dp;
+    }
+
     @Override
     public void onAdLoaded() {
         Log.w("AdMob", "BannerAdLoaded");
         if (executor.shouldAutoShow() && !executor.bannerVisible) {
             executor.showAd(true, null);
         }
-        executor.fireAdEvent("admob.banner.events.LOAD");
-        executor.fireAdEvent("onReceiveAd");
+
+        AdView view = executor.getAdView();
+        view.getViewTreeObserver().addOnGlobalLayoutListener(new ViewTreeObserver.OnGlobalLayoutListener() {
+            @Override
+            public void onGlobalLayout() {
+                JSONObject data = new JSONObject();
+                try {
+                    view.getViewTreeObserver().removeOnGlobalLayoutListener(this);
+                    int heightPx = view.getHeight(); //height is ready
+                    int heightDp = pxToDp(heightPx);
+
+                    data.put("bannerHeight", heightDp);
+                } catch (JSONException e) {
+                    e.printStackTrace();
+                }
+
+                executor.fireAdEvent("admob.banner.events.LOAD", data);
+                executor.fireAdEvent("onReceiveAd");
+            }
+        });
+
     }
 
     @Override