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

media/base/android/media_player_listener.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 "media/base/android/media_player_listener.h"
 
 #include "base/android/jni_android.h"
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop_proxy.h"
@@ skipping 17 matching lines @@
   DCHECK(message_loop_);
   DCHECK(media_player_);
 }
 
 MediaPlayerListener::~MediaPlayerListener() {}
 
 void MediaPlayerListener::CreateMediaPlayerListener(
     jobject context, jobject media_player) {
   JNIEnv* env = AttachCurrentThread();
   CHECK(env);
+  j_media_player_listener_.Reset(
+      Java_MediaPlayerListener_create(
+          env, reinterpret_cast<intptr_t>(this), context, media_player));
+}
 
-  Java_MediaPlayerListener_create(
-      env, reinterpret_cast<intptr_t>(this), context, media_player);
+
+void MediaPlayerListener::ReleaseMediaPlayerListenerResources() {
+  JNIEnv* env = AttachCurrentThread();
+  DCHECK(env);
+  if (env && !j_media_player_listener_.is_null()) {

[Yaron, 2012/11/27 05:58:00]

Just change the DCHECK above to CHECK and remove the "env" check

[Miguel Garcia, 2012/11/27 17:24:06]

Done.

+    Java_MediaPlayerListener_releaseResources(
+        env, j_media_player_listener_.obj());
+  }
+  j_media_player_listener_.Reset();
 }
 
 void MediaPlayerListener::OnMediaError(
     JNIEnv* /* env */, jobject /* obj */, jint error_type) {
   message_loop_->PostTask(FROM_HERE, base::Bind(
       &MediaPlayerBridge::OnMediaError, media_player_, error_type));
 }
 
 void MediaPlayerListener::OnVideoSizeChanged(
     JNIEnv* /* env */, jobject /* obj */, jint width, jint height) {
@@ skipping 19 matching lines @@
   message_loop_->PostTask(FROM_HERE, base::Bind(
       &MediaPlayerBridge::OnSeekComplete, media_player_));
 }
 
 void MediaPlayerListener::OnMediaPrepared(
     JNIEnv* /* env */, jobject /* obj */) {
   message_loop_->PostTask(FROM_HERE, base::Bind(
       &MediaPlayerBridge::OnMediaPrepared, media_player_));
 }
 
+void MediaPlayerListener::OnMediaInterrupted(
+    JNIEnv* /* env */, jobject /* obj */) {
+  message_loop_->PostTask(FROM_HERE, base::Bind(
+      &MediaPlayerBridge::OnMediaInterrupted, media_player_));
+}
+
 bool MediaPlayerListener::RegisterMediaPlayerListener(JNIEnv* env) {
   bool ret = RegisterNativesImpl(env);
   DCHECK(g_MediaPlayerListener_clazz);
   return ret;
 }
 
 }  // namespace media