diff options
| author | Jean-Michel Trivi <jmtrivi@google.com> | 2013-07-03 11:11:54 -0700 |
|---|---|---|
| committer | Jean-Michel Trivi <jmtrivi@google.com> | 2013-07-22 17:47:11 -0700 |
| commit | f47634edfe08fcebbf564a5df12a0a96906032c3 (patch) | |
| tree | e7c0d35c90473be357d6690dd7424599bbe010ff /media/java | |
| parent | d1233575b558d3dc585f7e04ff1377c24f3beaf4 (diff) | |
| download | frameworks_base-f47634edfe08fcebbf564a5df12a0a96906032c3.zip frameworks_base-f47634edfe08fcebbf564a5df12a0a96906032c3.tar.gz frameworks_base-f47634edfe08fcebbf564a5df12a0a96906032c3.tar.bz2 | |
Fix dump formatting of RemoteControlClient playback state dump
Take into account the two types of undefined plyback position.
Only use "ms" unit when there is a position to write.
Fix misplaced ",".
Change-Id: I5033ec6ee7480f1ac525af7b7b0a9da259614783
Diffstat (limited to 'media/java')
| -rw-r--r-- | media/java/android/media/MediaFocusControl.java | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/media/java/android/media/MediaFocusControl.java b/media/java/android/media/MediaFocusControl.java index 37c9e4a..f360cf1 100644 --- a/media/java/android/media/MediaFocusControl.java +++ b/media/java/android/media/MediaFocusControl.java @@ -974,10 +974,17 @@ public class MediaFocusControl implements OnFinished { @Override public String toString() { - return stateToString() + ", " - + ((mPositionMs == RemoteControlClient.PLAYBACK_POSITION_INVALID) ? - "PLAYBACK_POSITION_INVALID ," : String.valueOf(mPositionMs)) + "ms ," - + mSpeed + "X"; + return stateToString() + ", " + posToString() + ", " + mSpeed + "X"; + } + + private String posToString() { + if (mPositionMs == RemoteControlClient.PLAYBACK_POSITION_INVALID) { + return "PLAYBACK_POSITION_INVALID"; + } else if (mPositionMs == RemoteControlClient.PLAYBACK_POSITION_ALWAYS_UNKNOWN) { + return "PLAYBACK_POSITION_ALWAYS_UNKNOWN"; + } else { + return (String.valueOf(mPositionMs) + "ms"); + } } private String stateToString() { |