Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4dbd501c authored by Varun Berry's avatar Varun Berry Committed by Automerger Merge Worker
Browse files

Merge "Remove PhoneAccountHandle when building the intent to call Voicemail."...

Merge "Remove PhoneAccountHandle when building the intent to call Voicemail." into qt-dev am: 91bd2090 am: ef8a5e99 am: f2b7f43a am: 75dc1dea am: 23412cb0

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Dialer/+/16554823

Change-Id: I699a73e071562cae6be7362b7e065ab7caf8b953
parents cc648e95 23412cb0
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -560,7 +560,7 @@ public final class CallLogListItemViewHolder extends RecyclerView.ViewHolder
      case CallbackAction.VOICE:
        if (callLogCache.isVoicemailNumber(accountHandle, number)) {
          // Call to generic voicemail number, in case there are multiple accounts
          primaryActionButtonView.setTag(IntentProvider.getReturnVoicemailCallIntentProvider(null));
          primaryActionButtonView.setTag(IntentProvider.getReturnVoicemailCallIntentProvider());
        } else if (canSupportAssistedDialing()) {
          primaryActionButtonView.setTag(
              IntentProvider.getAssistedDialIntentProvider(
+2 −3
Original line number Diff line number Diff line
@@ -161,14 +161,13 @@ public abstract class IntentProvider {
    };
  }

  public static IntentProvider getReturnVoicemailCallIntentProvider(
      @Nullable PhoneAccountHandle phoneAccountHandle) {
  public static IntentProvider getReturnVoicemailCallIntentProvider() {
    return new IntentProvider() {
      @Override
      public Intent getIntent(Context context) {
        return PreCall.getIntent(
            context,
            CallIntentBuilder.forVoicemail(phoneAccountHandle, CallInitiationType.Type.CALL_LOG));
            CallIntentBuilder.forVoicemail(CallInitiationType.Type.CALL_LOG));
      }
    };
  }
+2 −2
Original line number Diff line number Diff line
@@ -117,10 +117,10 @@ public class CallIntentBuilder implements Parcelable {
  }

  public static CallIntentBuilder forVoicemail(
      @Nullable PhoneAccountHandle phoneAccountHandle, CallInitiationType.Type callInitiationType) {
      CallInitiationType.Type callInitiationType) {
    return new CallIntentBuilder(
            Uri.fromParts(PhoneAccount.SCHEME_VOICEMAIL, "", null), callInitiationType)
        .setPhoneAccountHandle(phoneAccountHandle);
        .setPhoneAccountHandle(null);
  }

  public void setUri(@NonNull Uri uri) {
+1 −1
Original line number Diff line number Diff line
@@ -72,7 +72,7 @@ public class CallCommand implements Command {
    CallIntentBuilder callIntentBuilder;
    if ("voicemail".equals(number)) {
      callIntentBuilder =
          CallIntentBuilder.forVoicemail(phoneAccountHandle, CallInitiationType.Type.DIALPAD);
          CallIntentBuilder.forVoicemail(CallInitiationType.Type.DIALPAD);
    } else {
      callIntentBuilder = new CallIntentBuilder(number, CallInitiationType.Type.DIALPAD);
    }
+1 −1
Original line number Diff line number Diff line
@@ -1136,7 +1136,7 @@ public class DialpadFragment extends Fragment

  public void callVoicemail() {
    PreCall.start(
        getContext(), CallIntentBuilder.forVoicemail(null, CallInitiationType.Type.DIALPAD));
        getContext(), CallIntentBuilder.forVoicemail(CallInitiationType.Type.DIALPAD));
    hideAndClearDialpad();
  }

Loading