Refactor RenderProcessHost to use IPC::Listener instead of RenderWidgetHost

content/browser/renderer_host/media/web_contents_video_capture_device.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.
 //
 // Implementation notes: This needs to work on a variety of hardware
 // configurations where the speed of the CPU and GPU greatly affect overall
 // performance. Spanning several threads, the process of capturing has been
 // split up into four conceptual stages:
 //
 //   1. Reserve Buffer: Before a frame can be captured, a slot in the consumer's
@@ skipping 855 matching lines @@
 }
 
 RenderWidgetHost* CaptureMachine::GetTarget() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   if (!web_contents())
     return NULL;
 
   RenderWidgetHost* rwh = NULL;
   if (fullscreen_widget_id_ != MSG_ROUTING_NONE) {
     RenderProcessHost* process = web_contents()->GetRenderProcessHost();
-    rwh = process ? process->GetRenderWidgetHostByID(fullscreen_widget_id_)
-                  : NULL;
+    if (process) {
+      rwh = RenderWidgetHost::FromID(process->GetID(), fullscreen_widget_id_);
+    } else {
+      rwh = NULL;

[jam, 2013/06/12 19:59:59]

nit: skip the else, it's not needed since rwh is initialized to NULL

[nasko, 2013/06/12 21:18:59]

Done.

+    }
   } else {
     rwh = web_contents()->GetRenderViewHost();
   }
 
   return rwh;
 }
 
 void CaptureMachine::DidCopyFromBackingStore(
     const base::Time& start_time,
     const scoped_refptr<media::VideoFrame>& target,
@@ skipping 376 matching lines @@
 void WebContentsVideoCaptureDevice::DeAllocate() {
   impl_->DeAllocate();
 }
 
 const media::VideoCaptureDevice::Name&
 WebContentsVideoCaptureDevice::device_name() {
   return device_name_;
 }
 
 }  // namespace content