Begin adding support for tab mirroring via the MediaStream audio/video capturing

content/browser/renderer_host/media/video_capture_host.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/browser/renderer_host/media/video_capture_host.h"
 
 #include "base/bind.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/stl_util.h"
 #include "content/browser/browser_main_loop.h"
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 #include "content/browser/renderer_host/media/video_capture_manager.h"
 #include "content/common/media/video_capture_messages.h"
+#include "content/public/common/media_stream_request.h"
 
 using content::BrowserMainLoop;
 using content::BrowserMessageFilter;
 using content::BrowserThread;
 
 struct VideoCaptureHost::Entry {
   Entry(VideoCaptureController* controller)
       : controller(controller) {}
 
   ~Entry() {}
@@ skipping 248 matching lines @@
   if (controller) {
     controller->StopCapture(controller_id, this);
     GetVideoCaptureManager()->RemoveController(controller, this);
   }
   delete it->second;
   entries_.erase(controller_id);
 }
 
 media_stream::VideoCaptureManager* VideoCaptureHost::GetVideoCaptureManager() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
-  return BrowserMainLoop::GetMediaStreamManager()->video_capture_manager();
+  // TODO(miu): Confirm use of VideoCaptureHostMsg_* only applies to the
+  // physical video capture devices, and does not have anything to do with
+  // MediaStreamHostMsg_*.

[wjia(left Chromium), 2012/09/07 14:11:36]

VideoCaptureHost should also take care of MEDIA_VIDEO_TAB_CAPTURE type.
Otherwise, you need a new class to handle the StartCapture/StopCapture requests
from renderer process.

[miu, 2012/09/07 23:14:28]

This functionality isn't needed yet, since we only plan to do tab mirroring via
MediaStreamManager.  I changed the comment to reflect that.

On 2012/09/07 14:11:36, wjia wrote:

+  return BrowserMainLoop::GetMediaStreamManager()->GetVideoCaptureManager(
+      content::MEDIA_VIDEO_DEVICE_CAPTURE);
 }