[RemotePlayback] Stop remote playback when disableRemotePlayback is set

chrome/browser/media/android/remote/remote_media_player_bridge.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 "chrome/browser/media/android/remote/remote_media_player_bridge.h"
 
 #include <utility>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
@@ skipping 258 matching lines @@
 }
 
 void RemoteMediaPlayerBridge::RequestRemotePlaybackControl() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   JNIEnv* env = AttachCurrentThread();
   CHECK(env);
 
   Java_RemoteMediaPlayerBridge_requestRemotePlaybackControl(env, java_bridge_);
 }
 
+void RemoteMediaPlayerBridge::RequestRemotePlaybackStop() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  JNIEnv* env = AttachCurrentThread();
+  CHECK(env);

[dcheng, 2016/11/04 01:51:44]

Please don't use CHECK for this:
https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/c++.md...

[whywhat, 2016/11/04 15:01:06]

Done. Seems like everyone else is using DCHECK(env) these days.

+
+  Java_RemoteMediaPlayerBridge_requestRemotePlaybackStop(env, java_bridge_);
+}
+
 void RemoteMediaPlayerBridge::SetNativePlayer() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   JNIEnv* env = AttachCurrentThread();
   CHECK(env);
 
   Java_RemoteMediaPlayerBridge_setNativePlayer(env, java_bridge_);
 }
 
 void RemoteMediaPlayerBridge::OnPlayerCreated() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
@@ skipping 183 matching lines @@
   Java_RemoteMediaPlayerBridge_setCookies(
       env, java_bridge_, ConvertUTF8ToJavaString(env, cookies));
 }
 
 MediaPlayerAndroid* RemoteMediaPlayerBridge::GetLocalPlayer() {
   return static_cast<RemoteMediaPlayerManager*>(manager())->GetLocalPlayer(
       player_id());
 }
 
 }  // namespace remote_media