diff options
author | Ta-wei Yen <twyen@google.com> | 2015-10-21 11:27:45 -0700 |
---|---|---|
committer | Ta-wei Yen <twyen@google.com> | 2015-10-21 17:29:01 -0700 |
commit | 05b3215d886d8e7787b49bf2390a87725ffa0c23 (patch) | |
tree | ac039076a1a11a8f9d613ca1b8503b416236d849 /InCallUI/src/com/android/incallui/CallButtonFragment.java | |
parent | 7753b65fb379b5d6a87702ee7e2f8fb1098ebca2 (diff) |
Fixed highlighter stuck on dialpad while in call.
+ InCallActivity.showDialpadFragment() now returns a boolean to indicate if
changes are actually made.
Bug: 18930766
Change-Id: Ib00859a58b289f988606e6427b1d874a69231fbb
Diffstat (limited to 'InCallUI/src/com/android/incallui/CallButtonFragment.java')
-rw-r--r-- | InCallUI/src/com/android/incallui/CallButtonFragment.java | 24 |
1 files changed, 14 insertions, 10 deletions
diff --git a/InCallUI/src/com/android/incallui/CallButtonFragment.java b/InCallUI/src/com/android/incallui/CallButtonFragment.java index 0987cacc8..1d32d8f1d 100644 --- a/InCallUI/src/com/android/incallui/CallButtonFragment.java +++ b/InCallUI/src/com/android/incallui/CallButtonFragment.java @@ -51,6 +51,7 @@ public class CallButtonFragment extends BaseFragment<CallButtonPresenter, CallButtonPresenter.CallButtonUi> implements CallButtonPresenter.CallButtonUi, OnMenuItemClickListener, OnDismissListener, View.OnClickListener { + private static final int INVALID_INDEX = -1; private int mButtonMaxVisible; // The button is currently visible in the UI @@ -61,6 +62,7 @@ public class CallButtonFragment private static final int BUTTON_MENU = 3; public interface Buttons { + public static final int BUTTON_AUDIO = 0; public static final int BUTTON_MUTE = 1; public static final int BUTTON_DIALPAD = 2; @@ -153,7 +155,7 @@ public class CallButtonFragment mOverflowButton = (ImageButton) parent.findViewById(R.id.overflowButton); mOverflowButton.setOnClickListener(this); mManageVideoCallConferenceButton = (ImageButton) parent.findViewById( - R.id.manageVideoCallConferenceButton); + R.id.manageVideoCallConferenceButton); mManageVideoCallConferenceButton.setOnClickListener(this); return parent; } @@ -181,7 +183,7 @@ public class CallButtonFragment int id = view.getId(); Log.d(this, "onClick(View " + view + ", id " + id + ")..."); - switch(id) { + switch (id) { case R.id.audioButton: onAudioButtonClicked(); break; @@ -258,11 +260,11 @@ public class CallButtonFragment } ImageButton[] normalButtons = { - mSwapButton, - mChangeToVideoButton, - mAddCallButton, - mMergeButton, - mOverflowButton + mSwapButton, + mChangeToVideoButton, + mAddCallButton, + mMergeButton, + mOverflowButton }; for (ImageButton button : normalButtons) { @@ -472,7 +474,7 @@ public class CallButtonFragment } addToOverflowMenu(i, button, menu); } - } else if (visibility == BUTTON_HIDDEN){ + } else if (visibility == BUTTON_HIDDEN) { button.setVisibility(View.GONE); } } @@ -781,9 +783,11 @@ public class CallButtonFragment @Override public void displayDialpad(boolean value, boolean animate) { - mShowDialpadButton.setSelected(value); if (getActivity() != null && getActivity() instanceof InCallActivity) { - ((InCallActivity) getActivity()).showDialpadFragment(value, animate); + boolean changed = ((InCallActivity) getActivity()).showDialpadFragment(value, animate); + if (changed) { + mShowDialpadButton.setSelected(value); + } } } |