diff options
author | Tyler Gunn <tgunn@google.com> | 2015-08-07 02:21:26 +0000 |
---|---|---|
committer | Android Git Automerger <android-git-automerger@android.com> | 2015-08-07 02:21:26 +0000 |
commit | 42f5797134b67e2475e3b4af11013beefecfa642 (patch) | |
tree | f435d76c2b7288081b84d5798afb8b04de67ab52 /InCallUI | |
parent | 3880980437573dc11f67590bc5c333b5c26327ec (diff) | |
parent | c18ae245df056d1d9d57a8ae7fcf16a14cff2d73 (diff) |
am ecd03f67: am ff12cb26: am c3f68285: am af99da31: Merge "Show "Note sent" toast when starting outgoing call with a note." into mnc-dr-dev
* commit 'ecd03f678fa150b67c882aeb14178c0ada5af42c':
Show "Note sent" toast when starting outgoing call with a note.
Diffstat (limited to 'InCallUI')
-rw-r--r-- | InCallUI/res/values/strings.xml | 4 | ||||
-rw-r--r-- | InCallUI/src/com/android/incallui/CallCardFragment.java | 6 | ||||
-rw-r--r-- | InCallUI/src/com/android/incallui/CallCardPresenter.java | 26 | ||||
-rw-r--r-- | InCallUI/src/com/android/incallui/CallList.java | 4 |
4 files changed, 39 insertions, 1 deletions
diff --git a/InCallUI/res/values/strings.xml b/InCallUI/res/values/strings.xml index 15cba1f32..e59867cb3 100644 --- a/InCallUI/res/values/strings.xml +++ b/InCallUI/res/values/strings.xml @@ -460,4 +460,8 @@ <!-- Used to inform the user that a call was received via a number other than the primary phone number associated with their device. [CHAR LIMIT=16] --> <string name="child_number">via <xliff:g id="child_number" example="650-555-1212">%s</xliff:g></string> + + <!-- Used to inform the user that the note associated with an outgoing call has been sent. + [CHAR LIMIT=32] --> + <string name="note_sent">Note sent</string> </resources> diff --git a/InCallUI/src/com/android/incallui/CallCardFragment.java b/InCallUI/src/com/android/incallui/CallCardFragment.java index 23c0990cf..c2b969c98 100644 --- a/InCallUI/src/com/android/incallui/CallCardFragment.java +++ b/InCallUI/src/com/android/incallui/CallCardFragment.java @@ -47,6 +47,7 @@ import android.view.animation.AnimationUtils; import android.widget.ImageButton; import android.widget.ImageView; import android.widget.TextView; +import android.widget.Toast; import com.android.contacts.common.util.MaterialColorMapUtils.MaterialPalette; import com.android.contacts.common.widget.FloatingActionButtonController; @@ -1114,6 +1115,11 @@ public class CallCardFragment extends BaseFragment<CallCardPresenter, CallCardPr }); } + @Override + public void showNoteSentToast() { + Toast.makeText(getContext(), R.string.note_sent, Toast.LENGTH_LONG).show(); + } + public void onDialpadVisibilityChange(boolean isShown) { mIsDialpadShowing = isShown; updateFabPosition(); diff --git a/InCallUI/src/com/android/incallui/CallCardPresenter.java b/InCallUI/src/com/android/incallui/CallCardPresenter.java index 305948d46..00343fcf3 100644 --- a/InCallUI/src/com/android/incallui/CallCardPresenter.java +++ b/InCallUI/src/com/android/incallui/CallCardPresenter.java @@ -76,6 +76,7 @@ public class CallCardPresenter extends Presenter<CallCardPresenter.CallCardUi> private CallTimer mCallTimer; private Context mContext; private boolean mSpinnerShowing = false; + private boolean mHasShownToast = false; public static class ContactLookupCallback implements ContactInfoCacheCallback { private final WeakReference<CallCardPresenter> mCallCardPresenter; @@ -120,6 +121,12 @@ public class CallCardPresenter extends Presenter<CallCardPresenter.CallCardUi> // Call may be null if disconnect happened already. if (call != null) { mPrimary = call; + if (shouldShowNoteSentToast(mPrimary)) { + final CallCardUi ui = getUi(); + if (ui != null) { + ui.showNoteSentToast(); + } + } CallList.getInstance().addCallUpdateListener(call.getId(), this); // start processing lookups right away. @@ -213,6 +220,10 @@ public class CallCardPresenter extends Presenter<CallCardPresenter.CallCardUi> Call previousPrimary = mPrimary; mPrimary = primary; + if (primaryChanged && shouldShowNoteSentToast(primary)) { + ui.showNoteSentToast(); + } + // Refresh primary call information if either: // 1. Primary call changed. // 2. The call's ability to manage conference has changed. @@ -923,6 +934,20 @@ public class CallCardPresenter extends Presenter<CallCardPresenter.CallCardUi> return isIncomingOrWaiting && !TextUtils.isEmpty(call.getCallSubject()); } + /** + * Determines whether the "note sent" toast should be shown. It should be shown for a new + * outgoing call with a subject. + * + * @param call The call + * @return {@code true} if the toast should be shown, {@code false} otherwise. + */ + private boolean shouldShowNoteSentToast(Call call) { + return call != null && !TextUtils + .isEmpty(call.getTelecommCall().getDetails().getIntentExtras().getString( + TelecomManager.EXTRA_CALL_SUBJECT)) && + (call.getState() == Call.State.DIALING || call.getState() == Call.State.CONNECTING); + } + public interface CallCardUi extends Ui { void setVisible(boolean on); void setCallCardVisible(boolean visible); @@ -950,5 +975,6 @@ public class CallCardPresenter extends Presenter<CallCardPresenter.CallCardUi> boolean isCallSubjectVisible(); void animateForNewOutgoingCall(); void sendAccessibilityAnnouncement(); + void showNoteSentToast(); } } diff --git a/InCallUI/src/com/android/incallui/CallList.java b/InCallUI/src/com/android/incallui/CallList.java index fbcc1cc61..666ba9529 100644 --- a/InCallUI/src/com/android/incallui/CallList.java +++ b/InCallUI/src/com/android/incallui/CallList.java @@ -161,7 +161,9 @@ public class CallList { } /** - * Called when a single call has changed session modification state. + * Called when the last forwarded number changes for a call. With IMS, the last forwarded + * number changes due to a supplemental service notification, so it is not pressent at the + * start of the call. * * @param call The call. */ |