Removes stopPing from onSaveInstanceState()

- I don't think it makes sense to do any processing in
  onSaveInstanceState() as we are already stopping the ping in
  onWindowVisibilityChanged.

- onSaveInstanceState() should solely deal with saving app states
  anyway.

Bug: 11156202
Change-Id: If2b17bd10b455899703aa2c8f9d15f04a9bea22f

1 files changed, 27 insertions, 0 deletions

diff --git a/InCallUI/src/com/android/incallui/AnswerFragment.java b/InCallUI/src/com/android/incallui/AnswerFragment.java
index 1b249f29f..a53d8c9e0 100644
--- a/InCallUI/src/com/android/incallui/AnswerFragment.java
+++ b/InCallUI/src/com/android/incallui/AnswerFragment.java
@@ -79,14 +79,33 @@ public class AnswerFragment extends BaseFragment<AnswerPresenter, AnswerPresente
         mGlowpad = (GlowPadWrapper) inflater.inflate(R.layout.answer_fragment,
                 container, false);
 
+        Log.d(this, "Creating view for answer fragment ", this);
+        Log.d(this, "Created from activity", getActivity());
         mGlowpad.setAnswerListener(this);
 
         return mGlowpad;
     }
 
     @Override
+    public void onDestroyView() {
+        Log.d(this, "onDestroyView");
+        if (mGlowpad != null) {
+            mGlowpad.stopPing();
+            mGlowpad = null;
+        }
+        super.onDestroyView();
+    }
+
+    @Override
     public void showAnswerUi(boolean show) {
         getView().setVisibility(show ? View.VISIBLE : View.GONE);
+
+        Log.d(this, "Show answer UI: " + show);
+        if (show) {
+            mGlowpad.startPing();
+        } else {
+            mGlowpad.stopPing();
+        }
     }
 
     @Override
@@ -126,6 +145,14 @@ public class AnswerFragment extends BaseFragment<AnswerPresenter, AnswerPresente
 
         final AlertDialog.Builder builder = new AlertDialog.Builder(getActivity()).setCancelable(
                 true).setView(lv);
+        builder.setOnCancelListener(new DialogInterface.OnCancelListener() {
+            @Override
+            public void onCancel(DialogInterface dialogInterface) {
+                if (mGlowpad != null) {
+                    mGlowpad.startPing();
+                }
+            }
+        });
         mCannedResponsePopup = builder.create();
         mCannedResponsePopup.show();
     }