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

Commit 1aee66f5 authored by Tyler Gunn's avatar Tyler Gunn
Browse files

Fix videoStateToString for audio/paused state. 

The videoStateToString method erroneously assumed that if the videostate
was Audio - Paused, that it was audio-only.  Simple logic error to fix.

Test: Added new unit tests.
Bug: 35645334
Change-Id: Ia5ae4d05284b70ad4ba86b8e5d2c53ee8e68a1fa
parent 975be6c0

telecomm/java/android/telecom/VideoProfile.java

+4 −1
Original line number Diff line number Diff line
@@ -235,7 +235,7 @@ public class VideoProfile implements Parcelable { 
        StringBuilder sb = new StringBuilder();

        sb.append("Audio");



        if (isAudioOnly(videoState)) {

        if (videoState == STATE_AUDIO_ONLY) {

            sb.append(" Only");

        } else {

            if (isTransmissionEnabled(videoState)) {
@@ -256,6 +256,9 @@ public class VideoProfile implements Parcelable { 


    /**

     * Indicates whether the video state is audio only.

     * <p>

     * Note: Considers only whether either both the {@link #STATE_RX_ENABLED} or

     * {@link #STATE_TX_ENABLED} bits are off, but not {@link #STATE_PAUSED}.

     *

     * @param videoState The video state.

     * @return {@code True} if the video state is audio only, {@code false} otherwise.