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

Commit e8cd5919 authored by Jeff Davidson's avatar Jeff Davidson
Browse files

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
parent d2d35453

telephony/java/android/telephony/PhoneStateListener.java

+2 −2
Original line number Diff line number Diff line
@@ -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

    }