[Android] A prototype of the interactive media notification.

content/browser/media/android/browser_media_player_manager.cc

Index: content/browser/media/android/browser_media_player_manager.cc
diff --git a/content/browser/media/android/browser_media_player_manager.cc b/content/browser/media/android/browser_media_player_manager.cc
index 32e2c0455d4181a0829f1b32fc0d003788607945..20443ea119902f61fcface60ae118a6e8e87bf82 100644
--- a/content/browser/media/android/browser_media_player_manager.cc
+++ b/content/browser/media/android/browser_media_player_manager.cc
@@ -233,6 +233,7 @@ void BrowserMediaPlayerManager::OnMediaMetadataChanged(
 
 void BrowserMediaPlayerManager::OnPlaybackComplete(int player_id) {
   Send(new MediaPlayerMsg_MediaPlaybackCompleted(RoutingID(), player_id));
+
   MediaSession::Get(web_contents())->RemovePlayer(this, player_id);
 
   if (fullscreen_player_id_ == player_id)
@@ -529,7 +530,8 @@ void BrowserMediaPlayerManager::OnPause(
   if (player)
     player->Pause(is_media_related_action);
 
-  MediaSession::Get(web_contents())->RemovePlayer(this, player_id);
+  if (is_media_related_action)

[mlamouri (slow - plz ping), 2015/06/18 16:43:22]

When do you end up in that situation?

[whywhat, 2015/06/19 16:00:34]

So it happens when I go to the notification and pause a single player media
session. The player's ReleaseMediaResources is called since the tab is invisible
and player is in the paused state (caused by the notification). Then it would
get removed from the session and the notification would get hidden.

See
https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/m...
where this is called with is_media_related_action is false.

I don't think we need to remove the notification if the player's resources were
released for some reason - we should update the status of the player in the
notification though and let the user try to resume it. 

[mlamouri (slow - plz ping), 2015/06/22 14:34:17]

Ok.

+    MediaSession::Get(web_contents())->RemovePlayer(this, player_id);
 }
 
 void BrowserMediaPlayerManager::OnSetVolume(int player_id, double volume) {