[Android] Stop all media on undoable close.

content/browser/web_contents/web_contents_android.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/web_contents/web_contents_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/command_line.h"
 #include "base/json/json_writer.h"
 #include "base/logging.h"
 #include "content/browser/android/interstitial_page_delegate_android.h"
 #include "content/browser/frame_host/interstitial_page_impl.h"
+#include "content/browser/media/android/browser_media_player_manager.h"
 #include "content/browser/media/media_web_contents_observer.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/common/frame_messages.h"
 #include "content/common/input_messages.h"
 #include "content/common/view_messages.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/content_switches.h"
@@ skipping 153 matching lines @@
 
 void WebContentsAndroid::OnHide(JNIEnv* env, jobject obj) {
   web_contents_->WasHidden();
   PauseVideo();
 }
 
 void WebContentsAndroid::OnShow(JNIEnv* env, jobject obj) {
   web_contents_->WasShown();
 }
 
+void WebContentsAndroid::ReleaseMediaPlayers(JNIEnv* env, jobject jobj) {
+  RenderViewHostImpl* rvhi = static_cast<RenderViewHostImpl*>(
+        web_contents_->GetRenderViewHost());

[qinmin, 2014/08/25 18:57:12]

nit: -2 indent

[David Trainor- moved to gerrit, 2014/08/25 23:42:15]

Done.

+#if defined(ENABLE_BROWSER_CDMS)
+  if (rvhi && rvhi->GetMainFrame()) {

[qinmin, 2014/08/25 18:57:12]

nit: nested if statements. Consider rewriting:

if (!rvhi || !rvhi->GetMainFrame())
  return;

.......

[David Trainor- moved to gerrit, 2014/08/25 23:42:15]

Done.

+    BrowserMediaPlayerManager* manager =
+        rvhi->media_web_contents_observer()->GetMediaPlayerManager(
+            rvhi->GetMainFrame());
+    if (manager)
+      manager->ReleaseAllMediaPlayers();
+  }
+#endif // defined(ENABLE_BROWSER_CDMS)
+}
+
 void WebContentsAndroid::PauseVideo() {
   RenderViewHostImpl* rvhi = static_cast<RenderViewHostImpl*>(
       web_contents_->GetRenderViewHost());
   if (rvhi)
     rvhi->media_web_contents_observer()->PauseVideo();
 }
 
 void WebContentsAndroid::AddStyleSheetByURL(
     JNIEnv* env,
     jobject obj,
@@ skipping 155 matching lines @@
   ScopedJavaGlobalRef<jobject> j_callback;
   j_callback.Reset(env, callback);
   content::RenderFrameHost::JavaScriptResultCallback js_callback =
       base::Bind(&JavaScriptResultCallback, j_callback);
 
   web_contents_->GetMainFrame()->ExecuteJavaScript(
       ConvertJavaStringToUTF16(env, script), js_callback);
 }
 
 }  // namespace content