[RemotePlayback] Stop remote playback when disableRemotePlayback is set

chrome/browser/media/android/remote/remote_media_player_bridge.cc

Index: chrome/browser/media/android/remote/remote_media_player_bridge.cc
diff --git a/chrome/browser/media/android/remote/remote_media_player_bridge.cc b/chrome/browser/media/android/remote/remote_media_player_bridge.cc
index 62f6d07bcd326500ec5359017de0be9bebe1028f..03da0c2c1fc9bf088d7594682a54db951f7c4120 100644
--- a/chrome/browser/media/android/remote/remote_media_player_bridge.cc
+++ b/chrome/browser/media/android/remote/remote_media_player_bridge.cc
@@ -276,6 +276,14 @@ void RemoteMediaPlayerBridge::RequestRemotePlaybackControl() {
   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();