commit cf1e1332e0b6880b4776d0bab31506092163423f
author Jaewan Kim <jaewan@google.com> Tue Mar 20 00:02:23 2018 +0900
committer Jaewan Kim <jaewan@google.com> Fri Mar 23 15:16:10 2018 +0900
tree c75b4860ef1d076a4522e9cb34462a3affc5d90c
parent 694a72d3ef5f40aa06fb3b43f7bc52eeedd50fed

MediaController2: Remove MediaPlaylistAgent in the ControllerCallback

Bug: 75500592
Test: Run CTS with MediaComponents/runcts.sh
Change-Id: I81e73103130781b94a27d56c696144ba5e7a4a29

packages/MediaComponents/src/com/android/media/MediaController2Impl.java

diff --git a/packages/MediaComponents/src/com/android/media/MediaController2Impl.java b/packages/MediaComponents/src/com/android/media/MediaController2Impl.java
index 0091816..250f77b 100644
--- a/packages/MediaComponents/src/com/android/media/MediaController2Impl.java
+++ b/packages/MediaComponents/src/com/android/media/MediaController2Impl.java
@@ -862,8 +862,7 @@
             if (!mInstance.isConnected()) {
                 return;
             }
-            // TODO(jaewan): Fix public API not to take playlistAgent.
-            mCallback.onPlaylistChanged(mInstance, null, playlist, metadata);
+            mCallback.onPlaylistChanged(mInstance, playlist, metadata);
         });
     }
 
@@ -875,8 +874,7 @@
             if (!mInstance.isConnected()) {
                 return;
             }
-            // TODO(jaewan): Fix public API not to take playlistAgent.
-            mCallback.onPlaylistMetadataChanged(mInstance, null, metadata);
+            mCallback.onPlaylistMetadataChanged(mInstance, metadata);
         });
     }
 
@@ -888,8 +886,7 @@
             if (!mInstance.isConnected()) {
                 return;
             }
-            // TODO(jaewan): Fix public API not to take playlistAgent.
-            mCallback.onShuffleModeChanged(mInstance, null, shuffleMode);
+            mCallback.onShuffleModeChanged(mInstance, shuffleMode);
         });
     }
 
@@ -901,8 +898,7 @@
             if (!mInstance.isConnected()) {
                 return;
             }
-            // TODO(jaewan): Fix public API not to take playlistAgent.
-            mCallback.onRepeatModeChanged(mInstance, null, repeatMode);
+            mCallback.onRepeatModeChanged(mInstance, repeatMode);
         });
     }