Media Related changes for TabCapture API

chrome/browser/media/media_internals.cc

Index: chrome/browser/media/media_internals.cc
diff --git a/chrome/browser/media/media_internals.cc b/chrome/browser/media/media_internals.cc
index cca5c7650b2cfa132099abedff26a1649a81176b..fc6951be6cd452fb5c331f993e52758b5e60b478 100644
--- a/chrome/browser/media/media_internals.cc
+++ b/chrome/browser/media/media_internals.cc
@@ -74,6 +74,12 @@ void MediaInternals::OnCaptureDevicesOpened(
   media_stream_capture_indicator_->CaptureDevicesOpened(render_process_id,
                                                         render_view_id,
                                                         devices);
+
+  for (content::MediaStreamDevices::const_iterator it = devices.begin();
+    it != devices.end(); ++it) {
+    OnMediaRequestStateChange(render_process_id, render_view_id, *it,

[no longer working on chromium, 2012/10/04 18:41:49]

Do we send the StateChange notification for all the opened/closed devices?
And how the extensions figure out if those devices are what they are interested
in?

[justinlin, 2012/10/08 08:59:45]

The extension will listen to all the TAB_MEDIA type stream changes. Actually,
only VIDEO_TAB_MEDIA right now since audio is not ready yet and otherwise we
will get dupes. Moved this loop back into media_stream_manager.cc

+      content::MediaStreamRequest::STATE_OPENING);
+  }
 }
 
 void MediaInternals::OnCaptureDevicesClosed(
@@ -84,6 +90,24 @@ void MediaInternals::OnCaptureDevicesClosed(
   media_stream_capture_indicator_->CaptureDevicesClosed(render_process_id,
                                                         render_view_id,
                                                         devices);
+
+  for (content::MediaStreamDevices::const_iterator it = devices.begin();
+    it != devices.end(); ++it) {
+    OnMediaRequestStateChange(render_process_id, render_view_id, *it,
+      content::MediaStreamRequest::STATE_ERROR);
+  }
+}
+
+void MediaInternals::OnMediaRequestStateChange(

[no longer working on chromium, 2012/10/04 18:41:49]

OnMediaRequestStateChanged

[justinlin, 2012/10/08 08:59:45]

Done.

+    int render_process_id,
+    int render_view_id,
+    const content::MediaStreamDevice& device,
+    const content::MediaStreamRequest::RequestState state) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+  if (observers_.size()) {
+    FOR_EACH_OBSERVER(MediaInternalsObserver, observers_,
+                      OnRequestUpdate(device, state));
+  }
 }
 
 void MediaInternals::AddObserver(MediaInternalsObserver* observer) {