From f1d82761ee0a4a0bb32a565f01127534cc9ab7db Mon Sep 17 00:00:00 2001 From: Jean-Michel Trivi Date: Tue, 14 Feb 2017 14:24:37 -0800 Subject: [PATCH] Player activity notification: SoundPool not ready for activity reporting Some plumbing is missing to accurately report SoundPool's playback state, do not report it Test: run cts -m CtsMediaTestCases -t android.media.cts.AudioPlaybackConfigurationTest Bug: 35263297 Change-Id: I93389c87e5136912bfd11831b87d80495c3a1c3e --- .../com/android/server/audio/PlaybackActivityMonitor.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/services/core/java/com/android/server/audio/PlaybackActivityMonitor.java b/services/core/java/com/android/server/audio/PlaybackActivityMonitor.java index bf1018f6a079d..a95a627d54e6a 100644 --- a/services/core/java/com/android/server/audio/PlaybackActivityMonitor.java +++ b/services/core/java/com/android/server/audio/PlaybackActivityMonitor.java @@ -103,6 +103,10 @@ public final class PlaybackActivityMonitor final boolean change; synchronized(mPlayerLock) { final AudioPlaybackConfiguration apc = mPlayers.get(new Integer(piid)); + // FIXME SoundPool not ready for state reporting + if (apc.getPlayerType() == AudioPlaybackConfiguration.PLAYER_TYPE_JAM_SOUNDPOOL) { + return; + } if (checkConfigurationCaller(piid, apc, binderUid)) { //TODO add generation counter to only update to the latest state change = apc.handleStateEvent(event);