Media Remoting: Switch to remoting only when media is playing.

media/remoting/remoting_renderer_controller.cc

Index: media/remoting/remoting_renderer_controller.cc
diff --git a/media/remoting/remoting_renderer_controller.cc b/media/remoting/remoting_renderer_controller.cc
index 40506d8251839278c751f63123a2e2ee45a1cd8f..a5b6f2ae24acd35d73c4ca4e289267054035e8c9 100644
--- a/media/remoting/remoting_renderer_controller.cc
+++ b/media/remoting/remoting_renderer_controller.cc
@@ -204,6 +204,19 @@ bool RemotingRendererController::IsAudioCodecSupported() {
   }
 }
 
+void RemotingRendererController::OnPlaying() {
+  DCHECK(thread_checker_.CalledOnValidThread());
+
+  is_paused_ = false;
+  UpdateAndMaybeSwitch();
+}
+
+void RemotingRendererController::OnPaused() {
+  DCHECK(thread_checker_.CalledOnValidThread());
+
+  is_paused_ = true;
+}
+
 bool RemotingRendererController::ShouldBeRemoting() {
   DCHECK(thread_checker_.CalledOnValidThread());
 
@@ -265,6 +278,9 @@ void RemotingRendererController::UpdateAndMaybeSwitch() {
   if (remote_rendering_started_ == should_be_remoting)
     return;
 
+  if (should_be_remoting && is_paused_)

[miu, 2016/12/19 22:56:53]

Let's add a comment to explain why this is good for the user experience.

[xjz, 2016/12/20 00:12:36]

Done.

+    return;  // Only switches to remoting when media is playing.

[miu, 2016/12/19 22:56:53]

s/switches/switch/

[xjz, 2016/12/20 00:12:36]

Done.

+
   // Switch between local renderer and remoting renderer.
   remote_rendering_started_ = should_be_remoting;