Re-added OnChannelClosing in MediaStreamDispatcherHost to close open media devices.

content/browser/renderer_host/media/media_stream_dispatcher_host.cc

Index: content/browser/renderer_host/media/media_stream_dispatcher_host.cc
diff --git a/content/browser/renderer_host/media/media_stream_dispatcher_host.cc b/content/browser/renderer_host/media/media_stream_dispatcher_host.cc
index 7da5f9193aab4526143b0f0d96424740695e976f..0d3a7b2d1d5d7a873069a2881e35bd07becafae1 100644
--- a/content/browser/renderer_host/media/media_stream_dispatcher_host.cc
+++ b/content/browser/renderer_host/media/media_stream_dispatcher_host.cc
@@ -57,17 +57,15 @@ void MediaStreamDispatcherHost::OnChannelClosing() {
   BrowserMessageFilter::OnChannelClosing();
   DVLOG(1) << "MediaStreamDispatcherHost::OnChannelClosing";
 
-  // TODO(mflodman) Remove this temporary solution when shut-down issue is
-  // resolved, i.e. uncomment the code below.
-  // Since the IPC channel is gone, close all requested VideCaptureDevices and
-  // cancel pending requests.
-//  manager()->CancelRequests(this);
-//  for (StreamMap::iterator it = streams_.begin();
-//       it != streams_.end();
-//       it++) {
-//    std::string label = it->first;
-//    manager()->StopGeneratedStream(label);
-//  }
+  // Since the IPC channel is gone, cancel pending requests and close all
+  // requested VideCaptureDevices.
+  manager()->CancelRequests(this);
+  for (StreamMap::iterator it = streams_.begin();
+       it != streams_.end();
+       it++) {
+    std::string label = it->first;

[tommi (sloooow) - chröme, 2012/02/23 10:03:22]

no need to create an extra string copy. should just be:
manager()->StopGeneratedStream(it->first);

+    manager()->StopGeneratedStream(label);
+  }
 }
 
 void MediaStreamDispatcherHost::OnGenerateStream(