[Android] A prototype of the interactive media notification.

content/browser/media/android/browser_media_player_manager.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/media/android/browser_media_player_manager.h"
 
 #include "base/android/scoped_java_ref.h"
 #include "base/command_line.h"
 #include "content/browser/android/content_view_core_impl.h"
 #include "content/browser/android/media_players_observer.h"
@@ skipping 215 matching lines @@
     int player_id, base::TimeDelta duration, int width, int height,
     bool success) {
   Send(new MediaPlayerMsg_MediaMetadataChanged(
       RoutingID(), player_id, duration, width, height, success));
   if (fullscreen_player_id_ == player_id)
     video_view_->UpdateMediaMetadata();
 }
 
 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)
     video_view_->OnPlaybackComplete();
 }
 
 void BrowserMediaPlayerManager::OnMediaInterrupted(int player_id) {
   // Tell WebKit that the audio should be paused, then release all resources
   Send(new MediaPlayerMsg_MediaPlayerReleased(RoutingID(), player_id));
   OnReleaseResources(player_id);
@@ skipping 276 matching lines @@
     player->SeekTo(time);
 }
 
 void BrowserMediaPlayerManager::OnPause(
     int player_id,
     bool is_media_related_action) {
   MediaPlayerAndroid* player = GetPlayer(player_id);
   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) {
   MediaPlayerAndroid* player = GetPlayer(player_id);
   if (player)
     player->SetVolume(volume);
 }
 
 void BrowserMediaPlayerManager::OnSetPoster(int player_id, const GURL& url) {
   // To be overridden by subclasses.
@@ skipping 103 matching lines @@
     external_video_surface_container_->ReleaseExternalVideoSurface(player_id);
 #endif  // defined(VIDEO_HOLE)
 }
 
 void BrowserMediaPlayerManager::ReleasePlayer(MediaPlayerAndroid* player) {
   player->Release();
   ReleaseMediaResources(player->player_id());
 }
 
 }  // namespace content