Android: not to pause screen capture when Chrome is put to background

content/renderer/render_view_impl.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 "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 #include <memory>
 
@@ skipping 66 matching lines @@
 #include "content/renderer/browser_plugin/browser_plugin_manager.h"
 #include "content/renderer/dom_storage/webstoragenamespace_impl.h"
 #include "content/renderer/drop_data_builder.h"
 #include "content/renderer/gpu/render_widget_compositor.h"
 #include "content/renderer/history_serialization.h"
 #include "content/renderer/idle_user_detector.h"
 #include "content/renderer/ime_event_guard.h"
 #include "content/renderer/input/input_handler_manager.h"
 #include "content/renderer/internal_document_state_data.h"
 #include "content/renderer/media/audio_device_factory.h"
+#include "content/renderer/media/media_stream_dispatcher.h"
 #include "content/renderer/media/video_capture_impl_manager.h"
 #include "content/renderer/navigation_state_impl.h"
 #include "content/renderer/render_frame_impl.h"
 #include "content/renderer/render_frame_proxy.h"
 #include "content/renderer/render_process.h"
 #include "content/renderer/render_thread_impl.h"
 #include "content/renderer/render_widget_fullscreen_pepper.h"
 #include "content/renderer/renderer_webapplicationcachehost_impl.h"
 #include "content/renderer/resizing_mode_selector.h"
 #include "content/renderer/savable_resources.h"
@@ skipping 2173 matching lines @@
   // We need to grab a pointer to the doomed WebView before we destroy it.
   WebView* doomed = webview_;
   RenderWidget::Close();
   webview_ = nullptr;
   g_view_map.Get().erase(doomed);
   g_routing_id_view_map.Get().erase(GetRoutingID());
   RenderThread::Get()->Send(new ViewHostMsg_Close_ACK(GetRoutingID()));
 }
 
 void RenderViewImpl::OnPageWasHidden() {
 #if defined(OS_ANDROID) && BUILDFLAG(ENABLE_WEBRTC)

[miu, 2017/03/22 23:09:43]

Why does BUILDFLAG(ENABLE_WEBRTC) matter? This logic should be applicable to ALL
remoting technologies. Let's change this to:

  #if defined(OS_ANDROID)
    SuspendVideoCaptureDevices();
  #if BUILDFLAG(ENABLE_WEBRTC)
    ...speech recog stuff...
  #endif
  #endif

[braveyao, 2017/03/23 03:44:20]

Done.

-  RenderThreadImpl::current()->video_capture_impl_manager()->
-      SuspendDevices(true);
+  SuspendVideoCaptureDevices(true);
+
   if (speech_recognition_dispatcher_)
     speech_recognition_dispatcher_->AbortAllRecognitions();
 #endif
 
   if (webview()) {
     // TODO(lfg): It's not correct to defer the page visibility to the main
     // frame. Currently, this is done because the main frame may override the
     // visibility of the page when prerendering. In order to fix this,
     // prerendering must be made aware of OOPIFs. https://crbug.com/440544
     blink::WebPageVisibilityState visibilityState =
         GetMainRenderFrame() ? GetMainRenderFrame()->visibilityState()
                              : blink::WebPageVisibilityStateHidden;
     webview()->setVisibilityState(visibilityState, false);
   }
 }
 
 void RenderViewImpl::OnPageWasShown() {
 #if defined(OS_ANDROID) && BUILDFLAG(ENABLE_WEBRTC)

[miu, 2017/03/22 23:09:43]

Remove the BUILDFLAG(ENABLE_WEBRTC) part.

[braveyao, 2017/03/23 03:44:20]

Done.

-  RenderThreadImpl::current()->video_capture_impl_manager()->
-      SuspendDevices(false);
+  SuspendVideoCaptureDevices(false);
 #endif
 
   if (webview()) {
     blink::WebPageVisibilityState visibilityState =
         GetMainRenderFrame() ? GetMainRenderFrame()->visibilityState()
                              : blink::WebPageVisibilityStateVisible;
     webview()->setVisibilityState(visibilityState, false);
   }
 }
 
@@ skipping 248 matching lines @@
       handled = true;
       break;
     }
     case TAP_MULTIPLE_TARGETS_STRATEGY_NONE:
       // No-op.
       break;
   }
 
   return handled;
 }
+
+void RenderViewImpl::SuspendVideoCaptureDevices(bool suspend) {
+  if (!main_render_frame_)
+    return;
+
+  MediaStreamDispatcher* media_stream_dispatcher =
+      main_render_frame_->GetMediaStreamDispatcher();
+  if (media_stream_dispatcher) {

[miu, 2017/03/22 23:09:43]

nit:

  if (!media_stream_dispatcher)
    return;

  ... rest of code one indent level less...

[braveyao, 2017/03/23 03:44:20]

Done.

+    StreamDeviceInfoArray video_array;
+    media_stream_dispatcher->GetVideoCaptureDevices(&video_array);
+    for (const StreamDeviceInfo& device_info : video_array) {
+      if (suspend) {

[miu, 2017/03/22 23:09:43]

See comments in header file about just having two methods instead of this bool
and the extra run-time cost here.

[braveyao, 2017/03/23 03:44:20]

Done.

+        RenderThreadImpl::current()->video_capture_impl_manager()->Suspend(
+            device_info.session_id);
+      } else {
+        RenderThreadImpl::current()->video_capture_impl_manager()->Resume(
+            device_info.session_id);
+      }
+    }
+  }
+}
 #endif  // defined(OS_ANDROID)
 
 unsigned RenderViewImpl::GetLocalSessionHistoryLengthForTesting() const {
   return history_list_length_;
 }
 
 void RenderViewImpl::SetFocusAndActivateForTesting(bool enable) {
   if (enable) {
     if (has_focus())
       return;
@@ skipping 131 matching lines @@
                             INPUT_EVENT_ACK_STATE_NOT_CONSUMED);
   }
 
   std::unique_ptr<InputEventAck> ack(
       new InputEventAck(InputEventAckSource::MAIN_THREAD, input_event->type(),
                         INPUT_EVENT_ACK_STATE_NOT_CONSUMED));
   OnInputEventAck(std::move(ack));
 }
 
 }  // namespace content