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

Commit 675a81b7 authored by Xiangyu/Malcolm Chen's avatar Xiangyu/Malcolm Chen Committed by Automerger Merge Worker
Browse files

Merge "Fix bug that we use the return value of bind service incorrectly." am: 85399462 

Original change: https://android-review.googlesource.com/c/platform/frameworks/opt/telephony/+/1493879

Change-Id: I61107cbf12cb3c5a41a73c8366d4a471fdcebba0
parents db288e43 85399462

src/java/com/android/internal/telephony/SMSDispatcher.java

+2 −2
Original line number Diff line number Diff line
@@ -365,7 +365,7 @@ public abstract class SMSDispatcher extends Handler { 
        public void sendSmsByCarrierApp(String carrierPackageName,

                                        SmsSenderCallback senderCallback) {

            mSenderCallback = senderCallback;

            if (mCarrierMessagingServiceWrapper.bindToCarrierMessagingService(

            if (!mCarrierMessagingServiceWrapper.bindToCarrierMessagingService(

                    mContext, carrierPackageName, ()->onServiceReady())) {

                Rlog.e(TAG, "bindService() for carrier messaging service failed");

                mSenderCallback.onSendSmsComplete(
@@ -561,7 +561,7 @@ public abstract class SMSDispatcher extends Handler { 
        void sendSmsByCarrierApp(String carrierPackageName,

                                 MultipartSmsSenderCallback senderCallback) {

            mSenderCallback = senderCallback;

            if (mCarrierMessagingServiceWrapper.bindToCarrierMessagingService(

            if (!mCarrierMessagingServiceWrapper.bindToCarrierMessagingService(

                    mContext, carrierPackageName, ()->onServiceReady())) {

                Rlog.e(TAG, "bindService() for carrier messaging service failed");

                mSenderCallback.onSendMultipartSmsComplete(