Adding error signalling from device to VideocaptureManager to relay up to MediaStream and WebKit.

content/browser/renderer_host/media/video_capture_manager.h

Index: content/browser/renderer_host/media/video_capture_manager.h
===================================================================
--- content/browser/renderer_host/media/video_capture_manager.h	(revision 89207)
+++ content/browser/renderer_host/media/video_capture_manager.h	(working copy)
@@ -15,6 +15,7 @@
 #include "base/lazy_instance.h"
 #include "base/threading/thread.h"
 #include "content/browser/renderer_host/media/media_stream_provider.h"
+#include "content/common/media/media_stream_options.h"
 #include "media/video/capture/video_capture_device.h"
 #include "media/video/capture/video_capture_types.h"
 
@@ -32,15 +33,15 @@
   static VideoCaptureManager* Get();
 
   // Implements MediaStreamProvider.
-  virtual bool Register(MediaStreamProviderListener* listener);
+  virtual void Register(MediaStreamProviderListener* listener);
 
   virtual void Unregister();
 
   virtual void EnumerateDevices();
 
-  virtual MediaCaptureSessionId Open(const MediaCaptureDeviceInfo& device);
+  virtual int Open(const StreamDeviceInfo& device);
 
-  virtual void Close(MediaCaptureSessionId capture_session_id);
+  virtual void Close(int capture_session_id);
 
   // Functions used to start and stop media flow.
   // Start allocates the device and no other application can use the device
@@ -54,6 +55,10 @@
   void Stop(const media::VideoCaptureSessionId capture_session_id,
             Task* stopped_task);
 
+  // A capture device error has occurred for |capture_sessions_id|. The device

[John Knottenbelt, 2011/06/16 15:20:43]

nit: capture_session_id 

[mflodman1, 2011/06/20 19:48:03]

Done.

+  // won't stream any more captured frames.
+  void Error(const media::VideoCaptureSessionId capture_session_id);

[scherkus (not reviewing), 2011/06/17 03:03:52]

const-ref ?

[mflodman1, 2011/06/20 19:48:03]

Done.
I'd prefer to remove this typedef and just use an int, but I'll discuss with Wei
and that will be another patch if agreed.

+
   virtual ~VideoCaptureManager();
 
   // Used by unit test to make sure a fake device is used instead of a real
@@ -69,34 +74,30 @@
 
   // Called by the public functions, executed on vc_device_thread_.
   void OnEnumerateDevices();
-  void OnOpen(MediaCaptureSessionId capture_session_id,
-              const MediaCaptureDeviceInfo device);
-  void OnClose(MediaCaptureSessionId capture_session_id);
+  void OnOpen(int capture_session_id, const StreamDeviceInfo device);

[John Knottenbelt, 2011/06/16 15:20:43]

Is it possible to pass const references here for StreamDeviceInfo, and also
below for PostOnDevicesEnumerated?

[mflodman1, 2011/06/20 19:48:03]

Done.

+  void OnClose(int capture_session_id);
   void OnStart(const media::VideoCaptureParams capture_params,
                media::VideoCaptureDevice::EventHandler* video_capture_receiver);
   void OnStop(const media::VideoCaptureSessionId capture_session_id,
               Task* stopped_task);
 
-
   // Executed on Browser::IO thread to call Listener.
-  void OnOpened(MediaCaptureSessionId capture_session_id);
-  void OnClosed(MediaCaptureSessionId capture_session_id);
-  void OnDevicesEnumerated(const MediaCaptureDevices& devices);
-  void OnError(MediaCaptureSessionId capture_session_id,
-               MediaStreamProviderError error);
+  void OnOpened(int capture_session_id);
+  void OnClosed(int capture_session_id);
+  void OnDevicesEnumerated(const StreamDeviceInfoArray& devices);
+  void OnError(int capture_session_id, MediaStreamProviderError error);
 
   // Executed on vc_device_thread_ to make sure Listener is called from
   // Browser::IO thread.
-  void PostOnOpened(MediaCaptureSessionId capture_session_id);
-  void PostOnClosed(MediaCaptureSessionId capture_session_id);
-  void PostOnDevicesEnumerated(MediaCaptureDevices devices);
-  void PostOnError(MediaCaptureSessionId capture_session_id,
-                   MediaStreamProviderError error);
+  void PostOnOpened(int capture_session_id);
+  void PostOnClosed(int capture_session_id);
+  void PostOnDevicesEnumerated(StreamDeviceInfoArray devices);
+  void PostOnError(int capture_session_id, MediaStreamProviderError error);
 
   // Helpers
   void GetAvailableDevices(media::VideoCaptureDevice::Names* device_names);
   bool DeviceOpened(const media::VideoCaptureDevice::Name& device_name);
-  bool DeviceOpened(const MediaCaptureDeviceInfo& device_info);
+  bool DeviceOpened(const StreamDeviceInfo& device_info);
   bool IsOnCaptureDeviceThread() const;
 
   // Thread for all calls to VideoCaptureDevice
@@ -104,7 +105,7 @@
 
   // Only accessed on Browser::IO thread
   MediaStreamProviderListener* listener_;
-  MediaCaptureSessionId new_capture_session_id_;
+  int new_capture_session_id_;
 
   // Only accessed from vc_device_thread_
   // TODO(mflodman) Change map key type when changing typedef for