refactor: use BaseFragment with LeakCanary RefWatcher

Kaushik Gopal · Kaushik Gopal · commit 835e28803b8d · 2015-05-12T11:21:17.000-07:00
diff --git a/app/src/main/java/com/morihacky/android/rxjava/BaseFragment.java b/app/src/main/java/com/morihacky/android/rxjava/BaseFragment.java
@@ -0,0 +1,15 @@
+package com.morihacky.android.rxjava;
+
+import android.support.v4.app.Fragment;
+import com.squareup.leakcanary.RefWatcher;
+
+public class BaseFragment
+      extends Fragment {
+
+    @Override
+    public void onDestroy() {
+        super.onDestroy();
+        RefWatcher refWatcher = MyApp.getRefWatcher();
+        refWatcher.watch(this);
+    }
+}
diff --git a/app/src/main/java/com/morihacky/android/rxjava/BufferDemoFragment.java b/app/src/main/java/com/morihacky/android/rxjava/BufferDemoFragment.java
@@ -34,7 +34,7 @@
  * as demonstrated in the case of {@link com.morihacky.android.rxjava.SubjectDebounceSearchEmitterFragment}
  */
 public class BufferDemoFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @InjectView(R.id.list_threading_log) ListView _logsList;
   @InjectView(R.id.btn_start_operation) Button _tapBtn;
diff --git a/app/src/main/java/com/morihacky/android/rxjava/ConcurrencyWithSchedulersDemoFragment.java b/app/src/main/java/com/morihacky/android/rxjava/ConcurrencyWithSchedulersDemoFragment.java
@@ -28,7 +28,7 @@
 import timber.log.Timber;
 
 public class ConcurrencyWithSchedulersDemoFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @InjectView(R.id.progress_operation_running) ProgressBar _progress;
   @InjectView(R.id.list_threading_log) ListView _logsList;
diff --git a/app/src/main/java/com/morihacky/android/rxjava/DemoTimeoutFragment.java b/app/src/main/java/com/morihacky/android/rxjava/DemoTimeoutFragment.java
@@ -28,7 +28,7 @@
 import timber.log.Timber;
 
 public class DemoTimeoutFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @InjectView(R.id.list_threading_log) ListView _logsList;
 
diff --git a/app/src/main/java/com/morihacky/android/rxjava/DoubleBindingTextViewFragment.java b/app/src/main/java/com/morihacky/android/rxjava/DoubleBindingTextViewFragment.java
@@ -20,7 +20,7 @@
 import rx.subjects.PublishSubject;
 
 public class DoubleBindingTextViewFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @InjectView(R.id.double_binding_num1) EditText _number1;
   @InjectView(R.id.double_binding_num2) EditText _number2;
diff --git a/app/src/main/java/com/morihacky/android/rxjava/FormValidationCombineLatestFragment.java b/app/src/main/java/com/morihacky/android/rxjava/FormValidationCombineLatestFragment.java
@@ -23,7 +23,7 @@
 import static com.google.common.base.Strings.isNullOrEmpty;
 
 public class FormValidationCombineLatestFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @InjectView(R.id.btn_demo_form_valid) TextView _btnValidIndicator;
   @InjectView(R.id.demo_combl_email) EditText _email;
diff --git a/app/src/main/java/com/morihacky/android/rxjava/MainFragment.java b/app/src/main/java/com/morihacky/android/rxjava/MainFragment.java
@@ -12,7 +12,7 @@
 import com.morihacky.android.rxjava.rxbus.RxBusDemoFragment;
 
 public class MainFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @Override
   public View onCreateView(LayoutInflater inflater,
diff --git a/app/src/main/java/com/morihacky/android/rxjava/PollingFragment.java b/app/src/main/java/com/morihacky/android/rxjava/PollingFragment.java
@@ -27,7 +27,7 @@
 import timber.log.Timber;
 
 public class PollingFragment
-    extends Fragment {
+    extends BaseFragment {
 
   public static final int INITIAL_DELAY = 0;
   public static final int POLLING_INTERVAL = 1000;
diff --git a/app/src/main/java/com/morihacky/android/rxjava/PseudoCacheConcatFragment.java b/app/src/main/java/com/morihacky/android/rxjava/PseudoCacheConcatFragment.java
@@ -30,7 +30,7 @@
 import static java.lang.String.format;
 
 public class PseudoCacheConcatFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @InjectView(R.id.log_list) ListView _resultList;
 
diff --git a/app/src/main/java/com/morihacky/android/rxjava/SubjectDebounceSearchEmitterFragment.java b/app/src/main/java/com/morihacky/android/rxjava/SubjectDebounceSearchEmitterFragment.java
@@ -46,7 +46,7 @@
  * incapable of debounce    : this is the primary reason, since creating new observable everytime in subscription disregards debounce on subsequent calls
  */
 public class SubjectDebounceSearchEmitterFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @InjectView(R.id.list_threading_log) ListView _logsList;
 
diff --git a/app/src/main/java/com/morihacky/android/rxjava/TimingDemoFragment.java b/app/src/main/java/com/morihacky/android/rxjava/TimingDemoFragment.java
@@ -19,7 +19,7 @@
 import timber.log.Timber;
 
 public class TimingDemoFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @Override
   public View onCreateView(LayoutInflater inflater,
diff --git a/app/src/main/java/com/morihacky/android/rxjava/rxbus/RxBusDemoFragment.java b/app/src/main/java/com/morihacky/android/rxjava/rxbus/RxBusDemoFragment.java
@@ -7,10 +7,11 @@
 import android.view.View;
 import android.view.ViewGroup;
 import butterknife.ButterKnife;
+import com.morihacky.android.rxjava.BaseFragment;
 import com.morihacky.android.rxjava.app.R;
 
 public class RxBusDemoFragment
-    extends Fragment {
+    extends BaseFragment {
 
   @Override
   public View onCreateView(LayoutInflater inflater,
diff --git a/app/src/main/java/com/morihacky/android/rxjava/rxbus/RxBusDemo_Bottom1Fragment.java b/app/src/main/java/com/morihacky/android/rxjava/rxbus/RxBusDemo_Bottom1Fragment.java
@@ -10,6 +10,7 @@
 import android.widget.TextView;
 import butterknife.ButterKnife;
 import butterknife.InjectView;
+import com.morihacky.android.rxjava.BaseFragment;
 import com.morihacky.android.rxjava.MainActivity;
 import com.morihacky.android.rxjava.app.R;
 import rx.functions.Action1;
@@ -18,7 +19,7 @@
 import static rx.android.app.AppObservable.bindFragment;
 
 public class RxBusDemo_Bottom1Fragment
-    extends Fragment {
+    extends BaseFragment {
 
   private RxBus _rxBus;
   private CompositeSubscription _subscriptions;
diff --git a/app/src/main/java/com/morihacky/android/rxjava/rxbus/RxBusDemo_Bottom2Fragment.java b/app/src/main/java/com/morihacky/android/rxjava/rxbus/RxBusDemo_Bottom2Fragment.java
@@ -10,6 +10,7 @@
 import android.widget.TextView;
 import butterknife.ButterKnife;
 import butterknife.InjectView;
+import com.morihacky.android.rxjava.BaseFragment;
 import com.morihacky.android.rxjava.MainActivity;
 import com.morihacky.android.rxjava.app.R;
 import java.util.List;
@@ -22,7 +23,7 @@
 import static rx.android.app.AppObservable.bindFragment;
 
 public class RxBusDemo_Bottom2Fragment
-    extends Fragment {
+    extends BaseFragment {
 
   private RxBus _rxBus;
   private CompositeSubscription _subscriptions;
diff --git a/app/src/main/java/com/morihacky/android/rxjava/rxbus/RxBusDemo_Bottom3Fragment.java b/app/src/main/java/com/morihacky/android/rxjava/rxbus/RxBusDemo_Bottom3Fragment.java
@@ -10,6 +10,7 @@
 import android.widget.TextView;
 import butterknife.ButterKnife;
 import butterknife.InjectView;
+import com.morihacky.android.rxjava.BaseFragment;
 import com.morihacky.android.rxjava.MainActivity;
 import com.morihacky.android.rxjava.app.R;
 import java.util.List;
@@ -24,7 +25,7 @@
 import static rx.android.app.AppObservable.bindFragment;
 
 public class RxBusDemo_Bottom3Fragment
-    extends Fragment {
+    extends BaseFragment {
 
   private RxBus _rxBus;
   private CompositeSubscription _subscriptions;
