From 3463ee10a247d52501ca1a5b4a272b812baf7dbe Mon Sep 17 00:00:00 2001 From: Jeff Davidson Date: Fri, 10 Nov 2017 15:58:20 -0800 Subject: Update PhoneStateListener#onCallStateChanged Javadoc. While the doc suggested that the phone number was "incoming" (e.g. only populated for incoming calls), it should also be populated for outgoing calls. Bug: 67748129 Test: Verified expected behavior on Taimen running OC-MR1, added CTS test (see other CL in this topic) Change-Id: I1ddd9c042f287953358a794b91b2af5ed6f5b597 Merged-In: I1ddd9c042f287953358a794b91b2af5ed6f5b597 --- telephony/java/android/telephony/PhoneStateListener.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/telephony/java/android/telephony/PhoneStateListener.java b/telephony/java/android/telephony/PhoneStateListener.java index 9ccfa942af5f..c7e51310e106 100644 --- a/telephony/java/android/telephony/PhoneStateListener.java +++ b/telephony/java/android/telephony/PhoneStateListener.java @@ -408,7 +408,7 @@ public class PhoneStateListener { /** * Callback invoked when device call state changes. * @param state call state - * @param incomingNumber incoming call phone number. If application does not have + * @param phoneNumber call phone number. If application does not have * {@link android.Manifest.permission#READ_PHONE_STATE READ_PHONE_STATE} permission, an empty * string will be passed as an argument. * @@ -416,7 +416,7 @@ public class PhoneStateListener { * @see TelephonyManager#CALL_STATE_RINGING * @see TelephonyManager#CALL_STATE_OFFHOOK */ - public void onCallStateChanged(int state, String incomingNumber) { + public void onCallStateChanged(int state, String phoneNumber) { // default implementation empty } -- cgit v1.2.3-59-g8ed1b