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

Commit a17bf2d0 authored by Hyundo Moon's avatar Hyundo Moon
Browse files

Move MediaMetadata class back to framework

Due to the @UnsupportedAppUsage on
MediaMetadata.getKeyFromMetadataEditorKey, we'd better not to move the
class to APEX. This CL moves the MediaMetadata back to framework.

Also, in order to avoid hidden framework API usages in mainline module,
this CL moves the usage of hidden constructor of MediaMetadata.Builder()
from MediaSessionEngine to MediaSession.

Bug: 123488217
Test: make update-api -j; make -j;
      vendor/google/tools/flashall;
      atest CtsMediaTestCases:android.media.cts.MediaSessionTest;
      atest CtsMediaTestCases:android.media.cts.MediaControllerTest;
Change-Id: I21b4f0f59214478071bb0eafa93bb9538e44bf4e
parent 9b314cea
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -3630,7 +3630,7 @@ package android.media.session {
  }
  public final class MediaSessionEngine implements java.lang.AutoCloseable {
    ctor public MediaSessionEngine(@NonNull android.content.Context, @NonNull android.media.session.SessionLink, @NonNull android.media.session.SessionCallbackLink, @NonNull android.media.session.MediaSessionEngine.CallbackStub, int);
    ctor public MediaSessionEngine(@NonNull android.content.Context, @NonNull android.media.session.SessionLink, @NonNull android.media.session.SessionCallbackLink, @NonNull android.media.session.MediaSessionEngine.CallbackStub);
    method public void close();
    method public String getCallingPackage();
    method @NonNull public android.media.session.MediaController getController();
+0 −1
Original line number Diff line number Diff line
@@ -66,7 +66,6 @@ filegroup {
filegroup {
    name: "media1-srcs",
    srcs: [
        "apex/java/android/media/MediaMetadata.java",
        "apex/java/android/media/MediaParceledListSlice.java",
        "apex/java/android/media/VolumeProvider.java",
        "apex/java/android/media/browse/MediaBrowser.java",
+1 −4
Original line number Diff line number Diff line
@@ -56,7 +56,6 @@ public final class MediaSessionEngine implements AutoCloseable {
    private static final String TAG = "MediaSession";

    private final Object mLock = new Object();
    private final int mMaxBitmapSize;

    private final MediaSession.Token mSessionToken;
    private final MediaController mController;
@@ -82,10 +81,9 @@ public final class MediaSessionEngine implements AutoCloseable {
     * @param cbStub A callback link that handles incoming command to {@link MediaSession.Callback}.
     */
    public MediaSessionEngine(@NonNull Context context, @NonNull SessionLink sessionLink,
            @NonNull SessionCallbackLink cbLink, @NonNull CallbackStub cbStub, int maxBitmapSize) {
            @NonNull SessionCallbackLink cbLink, @NonNull CallbackStub cbStub) {
        mSessionLink = sessionLink;
        mCbLink = cbLink;
        mMaxBitmapSize = maxBitmapSize;

        cbStub.setSessionImpl(this);
        mSessionToken = new MediaSession.Token(mSessionLink.getController());
@@ -346,7 +344,6 @@ public final class MediaSessionEngine implements AutoCloseable {
        int fields = 0;
        MediaDescription description = null;
        if (metadata != null) {
            metadata = (new MediaMetadata.Builder(metadata, mMaxBitmapSize)).build();
            if (metadata.containsKey(MediaMetadata.METADATA_KEY_DURATION)) {
                duration = metadata.getLong(MediaMetadata.METADATA_KEY_DURATION);
            }
Loading