Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(756)

Unified Diff: content/browser/renderer_host/media/video_capture_host.h

Issue 8304017: enable video capture to support sharing across multiple renderer processes (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/media/video_capture_host.h
===================================================================
--- content/browser/renderer_host/media/video_capture_host.h (revision 105502)
+++ content/browser/renderer_host/media/video_capture_host.h (working copy)
@@ -88,6 +88,12 @@
// between a VideCaptureMessageFilter and a VideoCaptureHost.
void OnStartCapture(int device_id,
const media::VideoCaptureParams& params);
+ void OnControllerAdded(
+ int device_id, const media::VideoCaptureParams& params,
+ VideoCaptureController* controller);
+ void DoControllerAddedOnIOThreead(
+ int device_id, const media::VideoCaptureParams params,
+ VideoCaptureController* controller);
// IPC message: Stop capture on device referenced by |device_id|.
void OnStopCapture(int device_id);
@@ -125,13 +131,20 @@
// Handle error coming from VideoCaptureDevice.
void DoHandleError(int device_id);
+ // Helpers.
+ media_stream::VideoCaptureManager* GetVideoCaptureManager();
+
typedef std::map<VideoCaptureControllerID,
scoped_refptr<VideoCaptureController> >EntryMap;
-
// A map of VideoCaptureControllerID to VideoCaptureController
// objects that is currently active.
EntryMap entries_;
+ typedef std::map<VideoCaptureControllerID,
+ media::VideoCapture::State>EntryState;
scherkus (not reviewing) 2011/10/19 18:02:21 nit: space between > and EntryState
wjia(left Chromium) 2011/10/21 00:56:13 Done.
+ // Record state of each VideoCaptureControllerID.
+ EntryState entry_state_;
perkj_chrome 2011/10/17 08:39:43 why two maps? Combine?
wjia(left Chromium) 2011/10/21 00:56:13 EntryState indicates that host has got StartCaptur
+
// Used to get a pointer to VideoCaptureManager to start/stop capture devices.
const content::ResourceContext* resource_context_;

Powered by Google App Engine
This is Rietveld 408576698