diff options
author | Christine Chen <christinech@google.com> | 2013-09-10 16:10:23 -0700 |
---|---|---|
committer | Christine Chen <christinech@google.com> | 2013-09-10 17:48:26 -0700 |
commit | 22c678c74fa38e921ea22b09962062fce6148047 (patch) | |
tree | 8d089677c24880f7acb1aca9f5b059772606b758 | |
parent | 066e3c2737dcc5288fec577324eb95dd94eb826e (diff) | |
download | packages_apps_InCallUI-22c678c74fa38e921ea22b09962062fce6148047.tar.gz packages_apps_InCallUI-22c678c74fa38e921ea22b09962062fce6148047.tar.bz2 packages_apps_InCallUI-22c678c74fa38e921ea22b09962062fce6148047.zip |
Adds onUiReady and onUiUnready to the presenter
- ConferenceManagerPresenter needs these two functions to listen to
CallList changes.
- Moves super.onUiUnready() to the top of the onUiUnready methods for
both the AnswerPresenter and ConferenceManagerPresenter classes.
Bug: 10696097
Change-Id: I050071f10ebe5112d9d41192ea8c78593189bd9b
-rw-r--r-- | src/com/android/incallui/AnswerPresenter.java | 1 | ||||
-rw-r--r-- | src/com/android/incallui/CallButtonPresenter.java | 5 | ||||
-rw-r--r-- | src/com/android/incallui/ConferenceManagerPresenter.java | 16 |
3 files changed, 18 insertions, 4 deletions
diff --git a/src/com/android/incallui/AnswerPresenter.java b/src/com/android/incallui/AnswerPresenter.java index e8bcc99c..e820929a 100644 --- a/src/com/android/incallui/AnswerPresenter.java +++ b/src/com/android/incallui/AnswerPresenter.java @@ -56,7 +56,6 @@ public class AnswerPresenter extends Presenter<AnswerPresenter.AnswerUi> if (mCallId != Call.INVALID_CALL_ID) { CallList.getInstance().removeCallUpdateListener(mCallId, this); } - } @Override diff --git a/src/com/android/incallui/CallButtonPresenter.java b/src/com/android/incallui/CallButtonPresenter.java index f14d851a..ec8e8447 100644 --- a/src/com/android/incallui/CallButtonPresenter.java +++ b/src/com/android/incallui/CallButtonPresenter.java @@ -50,13 +50,12 @@ public class CallButtonPresenter extends Presenter<CallButtonPresenter.CallButto @Override public void onUiUnready(CallButtonUi ui) { + super.onUiUnready(ui); + InCallPresenter.getInstance().removeListener(this); AudioModeProvider.getInstance().removeListener(this); mProximitySensor = null; - - // set Ui to null, so should go last - super.onUiUnready(ui); } @Override diff --git a/src/com/android/incallui/ConferenceManagerPresenter.java b/src/com/android/incallui/ConferenceManagerPresenter.java index ccda4a57..041614db 100644 --- a/src/com/android/incallui/ConferenceManagerPresenter.java +++ b/src/com/android/incallui/ConferenceManagerPresenter.java @@ -39,8 +39,24 @@ public class ConferenceManagerPresenter private Context mContext; @Override + public void onUiReady(ConferenceManagerUi ui) { + super.onUiReady(ui); + + // register for call state changes last + InCallPresenter.getInstance().addListener(this); + } + + @Override + public void onUiUnready(ConferenceManagerUi ui) { + super.onUiUnready(ui); + + InCallPresenter.getInstance().removeListener(this); + } + + @Override public void onStateChange(InCallState state, CallList callList) { if (getUi().isFragmentVisible()) { + Log.v(this, "onStateChange" + state); if (state == InCallState.INCALL && callList.getActiveOrBackgroundCall() != null && callList.getActiveOrBackgroundCall().isConferenceCall()) { Log.v(this, "Number of existing calls is " + |