diff options
author | Tyler Gunn <tgunn@google.com> | 2015-06-09 14:34:11 -0700 |
---|---|---|
committer | Tyler Gunn <tgunn@google.com> | 2015-06-09 14:34:11 -0700 |
commit | bc6f12ee42f810a68ef4d1e9185d42cd10124d9a (patch) | |
tree | 512d0e51045b8956508215ca51863a7c3e7e5e95 /telecomm | |
parent | aaf68f216327a8a2adaf72032904b50796b4aea7 (diff) | |
download | frameworks_base-bc6f12ee42f810a68ef4d1e9185d42cd10124d9a.zip frameworks_base-bc6f12ee42f810a68ef4d1e9185d42cd10124d9a.tar.gz frameworks_base-bc6f12ee42f810a68ef4d1e9185d42cd10124d9a.tar.bz2 |
Remove use of STATE_AUDIO_ONLY in a comparison.
- Cleanup javadoc to make it more explicit how to use STATE_AUDIO_ONLY.
- Use VideoProfile.isAudioOnly() instead of doing == comparison.
Bug: 21645094
Change-Id: I5a67290a996f8f338aebab72a4d3cb57f1b1993b
Diffstat (limited to 'telecomm')
-rw-r--r-- | telecomm/java/android/telecom/VideoProfile.java | 27 |
1 files changed, 26 insertions, 1 deletions
diff --git a/telecomm/java/android/telecom/VideoProfile.java b/telecomm/java/android/telecom/VideoProfile.java index 11a4976..dabf706 100644 --- a/telecomm/java/android/telecom/VideoProfile.java +++ b/telecomm/java/android/telecom/VideoProfile.java @@ -49,7 +49,32 @@ public class VideoProfile implements Parcelable { public static final int QUALITY_DEFAULT = 4; /** - * Call is currently in an audio-only mode with no video transmission or receipt. + * Used when answering or dialing a call to indicate that the call does not have a video + * component. + * <p> + * Should <b>not</b> be used in comparison checks to determine if a video state represents an + * audio-only call. + * <p> + * The following, for example, is not the correct way to check if a call is audio-only: + * <pre> + * {@code + * // This is the incorrect way to check for an audio-only call. + * if (videoState == VideoProfile.STATE_AUDIO_ONLY) { + * // Handle audio-only call. + * } + * } + * </pre> + * <p> + * Instead, use the {@link VideoProfile#isAudioOnly(int)} helper function to check if a + * video state represents an audio-only call: + * <pre> + * {@code + * // This is the correct way to check for an audio-only call. + * if (VideoProfile.isAudioOnly(videoState)) { + * // Handle audio-only call. + * } + * } + * </pre> */ public static final int STATE_AUDIO_ONLY = 0x0; |