commit 3cad49b83f0c3d35d783a3353ad7b7031ba4d79f
author linyuh <linyuh@google.com> Thu Mar 15 11:52:56 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Thu Mar 15 12:17:38 2018 -0700
tree 28c86d0363e170cdf443730913b89bc7ddd5352e
parent 8012611fdf40da5d8e66d4293304c10d1e185554

Register ShowBlockReportSpamDialogReceiver in MainActivity.

Bug: 73830902
Test: Manual
PiperOrigin-RevId: 189222303
Change-Id: I0397986d784d28e7948b39778319e3936f6c55e9

java/com/android/dialer/main/impl/MainActivity.java

diff --git a/java/com/android/dialer/main/impl/MainActivity.java b/java/com/android/dialer/main/impl/MainActivity.java
index fbcbeff..1d705f0 100644
--- a/java/com/android/dialer/main/impl/MainActivity.java
+++ b/java/com/android/dialer/main/impl/MainActivity.java
@@ -19,6 +19,8 @@
 import android.content.Context;
 import android.content.Intent;
 import android.os.Bundle;
+import android.support.v4.content.LocalBroadcastManager;
+import com.android.dialer.blockreportspam.ShowBlockReportSpamDialogReceiver;
 import com.android.dialer.common.Assert;
 import com.android.dialer.common.LogUtil;
 import com.android.dialer.configprovider.ConfigProviderBindings;
@@ -39,6 +41,12 @@
 
   private MainActivityPeer activePeer;
 
+  /**
+   * {@link android.content.BroadcastReceiver} that shows a dialog to block a number and/or report
+   * it as spam when notified.
+   */
+  private ShowBlockReportSpamDialogReceiver showBlockReportSpamDialogReceiver;
+
   public static Intent getShowCallLogIntent(Context context) {
     return getShowTabIntent(context, TabIndex.CALL_LOG);
   }
@@ -69,6 +77,8 @@
     // If peer was set by the super, don't reset it.
     activePeer = getNewPeer();
     activePeer.onActivityCreate(savedInstanceState);
+
+    showBlockReportSpamDialogReceiver = new ShowBlockReportSpamDialogReceiver(getFragmentManager());
   }
 
   protected MainActivityPeer getNewPeer() {
@@ -90,6 +100,10 @@
   protected void onResume() {
     super.onResume();
     activePeer.onActivityResume();
+
+    LocalBroadcastManager.getInstance(this)
+        .registerReceiver(
+            showBlockReportSpamDialogReceiver, ShowBlockReportSpamDialogReceiver.getIntentFilter());
   }
 
   @Override
@@ -102,6 +116,8 @@
   protected void onPause() {
     super.onPause();
     activePeer.onActivityPause();
+
+    LocalBroadcastManager.getInstance(this).unregisterReceiver(showBlockReportSpamDialogReceiver);
   }
 
   @Override

java/com/android/dialer/main/impl/NewMainActivityPeer.java

diff --git a/java/com/android/dialer/main/impl/NewMainActivityPeer.java b/java/com/android/dialer/main/impl/NewMainActivityPeer.java
index 081559a..9607879 100644
--- a/java/com/android/dialer/main/impl/NewMainActivityPeer.java
+++ b/java/com/android/dialer/main/impl/NewMainActivityPeer.java
@@ -22,8 +22,6 @@
 import android.support.v4.app.Fragment;
 import android.support.v4.app.FragmentManager;
 import android.support.v4.app.FragmentTransaction;
-import android.support.v4.content.LocalBroadcastManager;
-import com.android.dialer.blockreportspam.ShowBlockReportSpamDialogReceiver;
 import com.android.dialer.calllog.CallLogComponent;
 import com.android.dialer.calllog.ui.NewCallLogFragment;
 import com.android.dialer.common.concurrent.DefaultFutureCallback;
@@ -39,12 +37,9 @@
 public class NewMainActivityPeer implements MainActivityPeer {
 
   private final MainActivity mainActivity;
-  private final ShowBlockReportSpamDialogReceiver showBlockReportSpamDialogReceiver;
 
   public NewMainActivityPeer(MainActivity mainActivity) {
     this.mainActivity = mainActivity;
-    this.showBlockReportSpamDialogReceiver =
-        new ShowBlockReportSpamDialogReceiver(mainActivity.getFragmentManager());
   }
 
   @Override
@@ -59,20 +54,13 @@
   }
 
   @Override
-  public void onActivityResume() {
-    LocalBroadcastManager.getInstance(mainActivity)
-        .registerReceiver(
-            showBlockReportSpamDialogReceiver, ShowBlockReportSpamDialogReceiver.getIntentFilter());
-  }
+  public void onActivityResume() {}
 
   @Override
   public void onUserLeaveHint() {}
 
   @Override
-  public void onActivityPause() {
-    LocalBroadcastManager.getInstance(mainActivity)
-        .unregisterReceiver(showBlockReportSpamDialogReceiver);
-  }
+  public void onActivityPause() {}
 
   @Override
   public void onActivityStop() {}