Free (and pause) audio resources when getting a phone call

content/browser/android/media_player_manager_android.cc

 // Copyright (c) 2012 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/android/media_player_manager_android.h"
 
 #include "base/bind.h"
 #include "content/browser/android/cookie_getter_impl.h"
 #include "content/common/media/media_player_messages.h"
 #include "content/public/browser/browser_context.h"
@@ skipping 105 matching lines @@
                  base::Unretained(this)),
       base::Bind(&MediaPlayerManagerAndroid::OnBufferingUpdate,
                  base::Unretained(this)),
       base::Bind(&MediaPlayerManagerAndroid::OnPrepared,
                  base::Unretained(this)),
       base::Bind(&MediaPlayerManagerAndroid::OnPlaybackComplete,
                  base::Unretained(this)),
       base::Bind(&MediaPlayerManagerAndroid::OnSeekComplete,
                  base::Unretained(this)),
       base::Bind(&MediaPlayerManagerAndroid::OnTimeUpdate,
+                 base::Unretained(this)),
+      base::Bind(&MediaPlayerManagerAndroid::OnMediaInterrupted,
                  base::Unretained(this))));
 
   // Send a MediaPrepared message to webkit so that Load() can finish.
   Send(new MediaPlayerMsg_MediaPrepared(
       routing_id(), player_id, GetPlayer(player_id)->GetDuration()));
 }
 
 void MediaPlayerManagerAndroid::OnStart(int player_id) {
   MediaPlayerBridge* player = GetPlayer(player_id);
   if (player)
@@ skipping 77 matching lines @@
   if (fullscreen_player_id_ != -1)
     video_view_.UpdateMediaMetadata();
 }
 
 void MediaPlayerManagerAndroid::OnPlaybackComplete(int player_id) {
   Send(new MediaPlayerMsg_MediaPlaybackCompleted(routing_id(), player_id));
   if (fullscreen_player_id_ != -1)
     video_view_.OnPlaybackComplete();
 }
 
+void MediaPlayerManagerAndroid::OnMediaInterrupted(int player_id) {
+  // Tell WebKit that the audio should be paused, then release all resources
+  Send(new MediaPlayerMsg_DidMediaPlayerPause(
+      routing_id(), fullscreen_player_id_));

[qinmin, 2012/11/26 15:02:21]

should be player_id, or otherwise, it will be routed to a different
webmediaplayer

[Miguel Garcia, 2012/11/26 15:42:28]

Thanks for noticing, changed now.

On 2012/11/26 15:02:21, qinmin wrote:

+  OnReleaseResources(player_id);
+}
+
 void MediaPlayerManagerAndroid::OnBufferingUpdate(
     int player_id, int percentage) {
   Send(new MediaPlayerMsg_MediaBufferingUpdate(
       routing_id(), player_id, percentage));
   if (fullscreen_player_id_ != -1)
     video_view_.OnBufferingUpdate(percentage);
 }
 
 void MediaPlayerManagerAndroid::OnSeekComplete(int player_id,
                                                base::TimeDelta current_time) {
@@ skipping 52 matching lines @@
     }
   }
 }
 
 void MediaPlayerManagerAndroid::ReleaseMediaResources(
     MediaPlayerBridge* player) {
   // Nothing needs to be done.
 }
 
 }  // namespace content