commit 0939bc77ff1626e7204c00edbd5486d9744e888d
author Hall Liu <hallliu@google.com> Sat Jan 21 00:43:38 2017 +0000
committer Hall Liu <hallliu@google.com> Mon Jan 30 19:33:51 2017 +0000
tree e431864c4d7115a943291562d36bea48081e7377
parent 8647593466a4cbc47fe13706aaa3e922ef624385

Properly route the SET_CALL_FORWARD_DONE message

In ImsPhone, the message indicating that call forwarding was set was
being directed straight to ImsPhoneMmiCode instead of routing through
the handler in ImsPhone, which caused the call forwarding notification
to never get set. This sends the proper message to ImsPhone first, which
will put up the call forwarding notification.

Change-Id: I5d06a0bad6e2bf368817ec623905f4a16e06d358
Fix: 34185152
Test: manual (to test on US carriers, force useImsForUt in GsmCdmaPhone)

src/java/com/android/internal/telephony/imsphone/ImsPhone.java

diff --git a/src/java/com/android/internal/telephony/imsphone/ImsPhone.java b/src/java/com/android/internal/telephony/imsphone/ImsPhone.java
index edd2c9d..0e5fe5e 100644
--- a/src/java/com/android/internal/telephony/imsphone/ImsPhone.java
+++ b/src/java/com/android/internal/telephony/imsphone/ImsPhone.java
@@ -802,7 +802,7 @@
                         dialingNumber,
                         serviceClass,
                         timerSeconds,
-                        onComplete);
+                        resp);
             } catch (ImsException e) {
                 sendErrorResponse(onComplete, e);
             }