Merge pull request #4325 from ByteHamster/dispose-loader-before-null

Dispose loaders before setting controller to null

app/src/main/java/de/danoeh/antennapod/fragment/ChaptersFragment.java

@@ -93,17 +93,12 @@ public class ChaptersFragment extends Fragment {
     }
 
     @Override
-    public void onDestroyView() {
+    public void onStop() {
-        super.onDestroyView();
+        super.onStop();
 
         if (disposable != null) {
             disposable.dispose();
         }
-    }
-
-    @Override
-    public void onStop() {
-        super.onStop();
         controller.release();
         controller = null;
         EventBus.getDefault().unregister(this);

app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java

@@ -128,6 +128,10 @@ public class CoverFragment extends Fragment {
     @Override
     public void onStop() {
         super.onStop();
+
+        if (disposable != null) {
+            disposable.dispose();
+        }
         controller.release();
         controller = null;
         EventBus.getDefault().unregister(this);
@@ -203,15 +207,6 @@ public class CoverFragment extends Fragment {
         }
     }
 
-    @Override
-    public void onDestroyView() {
-        super.onDestroyView();
-
-        if (disposable != null) {
-            disposable.dispose();
-        }
-    }
-
     void onPlayPause() {
         if (controller == null) {
             return;

app/src/main/java/de/danoeh/antennapod/fragment/ItemDescriptionFragment.java

@@ -66,9 +66,6 @@ public class ItemDescriptionFragment extends Fragment {
     public void onDestroy() {
         super.onDestroy();
         Log.d(TAG, "Fragment destroyed");
-        if (webViewLoader != null) {
-            webViewLoader.dispose();
-        }
         if (webvDescription != null) {
             webvDescription.removeAllViews();
             webvDescription.destroy();
@@ -168,6 +165,10 @@ public class ItemDescriptionFragment extends Fragment {
     @Override
     public void onStop() {
         super.onStop();
+
+        if (webViewLoader != null) {
+            webViewLoader.dispose();
+        }
         controller.release();
         controller = null;
     }