ScreenCapture for Android phase1, part II

content/browser/renderer_host/media/peer_connection_tracker_host.cc

 // Copyright (c) 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/renderer_host/media/peer_connection_tracker_host.h"
 
 #include "base/power_monitor/power_monitor.h"
 #include "content/browser/media/webrtc/webrtc_eventlog_host.h"
 #include "content/browser/media/webrtc/webrtc_internals.h"
 #include "content/browser/renderer_host/render_process_host_impl.h"
+#include "content/common/media/media_stream_options.h"
 #include "content/common/media/peer_connection_tracker_messages.h"
 
 namespace content {
 
 PeerConnectionTrackerHost::PeerConnectionTrackerHost(
     int render_process_id,
     WebRTCEventLogHost* event_log_host)
     : BrowserMessageFilter(PeerConnectionTrackerMsgStart),
       render_process_id_(render_process_id),
+      isScreenCapture_(false),
       event_log_host_(event_log_host) {
   DCHECK(event_log_host);
 }
 
 bool PeerConnectionTrackerHost::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
 
   IPC_BEGIN_MESSAGE_MAP(PeerConnectionTrackerHost, message)
     IPC_MESSAGE_HANDLER(PeerConnectionTrackerHost_AddPeerConnection,
                         OnAddPeerConnection)
@@ skipping 75 matching lines @@
     bool video,
     const std::string& audio_constraints,
     const std::string& video_constraints) {
   WebRTCInternals::GetInstance()->OnGetUserMedia(render_process_id_,
                                                  peer_pid(),
                                                  origin,
                                                  audio,
                                                  video,
                                                  audio_constraints,
                                                  video_constraints);
+#if defined(OS_ANDROID)
+  isScreenCapture_ =
+      video_constraints.find(kMediaStreamSourceScreen) != std::string::npos;

[miu, 2016/07/14 22:25:53]

This feels fragile. Is the |video_constraints| string something that needs to be
parsed, because it contains formatted/serialized structured data?

[braveyao, 2016/07/18 20:46:31]

Done. Found a more solid method: parse the blink::WebUserMediaRequest in
peer_connection_tracker.cc and send the source type to here.

+#endif
 }
 
 void PeerConnectionTrackerHost::OnSuspend() {
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
       base::Bind(&PeerConnectionTrackerHost::SendOnSuspendOnUIThread, this));
 }
 
 void PeerConnectionTrackerHost::SendOnSuspendOnUIThread() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   content::RenderProcessHost* host =
       content::RenderProcessHost::FromID(render_process_id_);
-  if (host)
+  if (host && !isScreenCapture_)
     host->Send(new PeerConnectionTracker_OnSuspend());
 }
 
 }  // namespace content