Update media session metadata and show content text with artist and a…

…lbum

support/android/apk/servoapp/src/main/java/org/mozilla/servo/MainActivity.java

@@ -233,6 +233,7 @@ public void onMediaSessionMetadata(String title, String artist, String album) {
         mMediaSession = new MediaSession(mServoView, this, getApplicationContext());
       }
       Log.d("onMediaSessionMetadata", title + " " + artist + " " + album);
+      mMediaSession.updateMetadata(title, artist, album);
     }
 
     @Override
@@ -241,13 +242,16 @@ public void onMediaSessionPlaybackStateChange(int state) {
       if (mMediaSession == null) {
         mMediaSession = new MediaSession(mServoView, this, getApplicationContext());
       }
+
+      mMediaSession.setPlaybackState(state);
+
       if (state == MediaSession.PLAYBACK_STATE_NONE) {
           mMediaSession.hideMediaSessionControls();
           return;
       }
       if (state == MediaSession.PLAYBACK_STATE_PLAYING ||
           state == MediaSession.PLAYBACK_STATE_PAUSED) {
-          mMediaSession.showMediaSessionControls(state);
+          mMediaSession.showMediaSessionControls();
           return;
       }
     }

support/android/apk/servoapp/src/main/java/org/mozilla/servo/MediaSession.java

@@ -58,9 +58,16 @@ public int get() {
     ServoView mView;
     MainActivity mActivity;
     Context mContext;
+
     NotificationID mNotificationID;
     BroadcastReceiver mMediaSessionActionReceiver;
 
+    int mPlaybackState = PLAYBACK_STATE_PAUSED;
+
+    String mTitle;
+    String mArtist;
+    String mAlbum;
+
     public MediaSession(ServoView view, MainActivity activity, Context context) {
       mView = view;
       mActivity = activity;
@@ -85,13 +92,13 @@ private void createMediaNotificationChannel() {
       }
     }
 
-    public void showMediaSessionControls(int playbackState) {
-      Log.d("MediaSession", "showMediaSessionControls " + playbackState);
+    public void showMediaSessionControls() {
+      Log.d("MediaSession", "showMediaSessionControls " + mPlaybackState);
       IntentFilter filter = new IntentFilter();
-      if (playbackState == PLAYBACK_STATE_PAUSED) {
+      if (mPlaybackState == PLAYBACK_STATE_PAUSED) {
         filter.addAction(KEY_MEDIA_PLAY);
       }
-      if (playbackState == PLAYBACK_STATE_PLAYING) {
+      if (mPlaybackState == PLAYBACK_STATE_PLAYING) {
         filter.addAction(KEY_MEDIA_PAUSE);
       }
       filter.addAction(KEY_MEDIA_STOP);
@@ -129,20 +136,36 @@ public void onReceive(Context context, Intent intent) {
       Notification.Builder builder = new Notification.Builder(mContext, this.MEDIA_CHANNEL_ID);
       builder
         .setSmallIcon(R.drawable.media_session_icon)
-        .setContentTitle("This is the notification title")
+        .setContentTitle(mTitle)
         .setVisibility(Notification.VISIBILITY_PUBLIC)
         .addAction(stopAction)
         .setStyle(new Notification.MediaStyle());
 
-      if (playbackState == PLAYBACK_STATE_PAUSED) {
+      String contentText = new String();
+      if (mArtist != null && !mArtist.isEmpty()) {
+        contentText = mArtist;
+      }
+      if (mAlbum != null && !mAlbum.isEmpty()) {
+        if (!contentText.isEmpty()) {
+          contentText += " - " + mAlbum;
+        } else {
+          contentText = mAlbum;
+        }
+      }
+
+      if (!contentText.isEmpty()) {
+        builder.setContentText(contentText);
+      }
+
+      if (mPlaybackState == PLAYBACK_STATE_PAUSED) {
         Intent playIntent = new Intent(KEY_MEDIA_PLAY);
         Notification.Action playAction =
           new Notification.Action(R.drawable.media_session_play, "Play",
             PendingIntent.getBroadcast(mContext, 0, playIntent, 0));
         builder.addAction(playAction);
       }
 
-      if (playbackState == PLAYBACK_STATE_PLAYING) {
+      if (mPlaybackState == PLAYBACK_STATE_PLAYING) {
         Intent pauseIntent = new Intent(KEY_MEDIA_PAUSE);
         Notification.Action pauseAction =
           new Notification.Action(R.drawable.media_session_pause, "Pause",
@@ -163,4 +186,18 @@ public void hideMediaSessionControls() {
       mContext.unregisterReceiver(mMediaSessionActionReceiver);
       mMediaSessionActionReceiver = null;
     }
+
+    public void setPlaybackState(int state) {
+      mPlaybackState = state;
+    }
+
+    public void updateMetadata(String title, String artist, String album) {
+      mTitle = title;
+      mArtist = artist;
+      mAlbum = album;
+
+      if (mMediaSessionActionReceiver != null) {
+        showMediaSessionControls();
+      }
+    }
 }