Merge pull request #2 from collinjackson/flutter/plugins#849

[firebase_admob]Destroy adView Before contentView Checking
firebase · Aug 30, 2019 · 0e1d2fa · 0e1d2fa
2 parents cdbcf01 + 296cd49
commit 0e1d2fa
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/firebase_admob/android/src/main/java/io/flutter/plugins/firebaseadmob/MobileAd.java b/packages/firebase_admob/android/src/main/java/io/flutter/plugins/firebaseadmob/MobileAd.java
@@ -182,11 +182,11 @@ void show() {
     void dispose() {
       super.dispose();
 
+      adView.destroy();
+
       View contentView = activity.findViewById(id);
       if (contentView == null || !(contentView.getParent() instanceof ViewGroup)) return;
 
-      adView.destroy();
-
       ViewGroup contentParent = (ViewGroup) (contentView.getParent());
       contentParent.removeView(contentView);
     }