Delete MediaStreamManager in the same way as AudioManager

content/browser/renderer_host/media/media_stream_device_settings.cc

Index: content/browser/renderer_host/media/media_stream_device_settings.cc
diff --git a/content/browser/renderer_host/media/media_stream_device_settings.cc b/content/browser/renderer_host/media/media_stream_device_settings.cc
index c3a33506affb92e87aa25bf0c20d7cbf5a92e1ed..456c217d3afd1e35922cfdb47f6f8a00449cbaaa 100644
--- a/content/browser/renderer_host/media/media_stream_device_settings.cc
+++ b/content/browser/renderer_host/media/media_stream_device_settings.cc
@@ -43,15 +43,18 @@ class ResponseCallbackHelper
 
   void PostResponse(const std::string& label,
                     const content::MediaStreamDevices& devices) {
+    if (!settings_)

[tommi (sloooow) - chröme, 2012/07/18 08:15:23]

it's better to do this check only on the IO thread as it was before.  This
change makes it so that we're checking the variable on more than one thread.

[no longer working on chromium, 2012/07/18 09:39:28]

But can BrowserThread::PostTask(BrowserThread::IO, .. crash when the IO thread
has gone away?

+      return;
+
     if (!BrowserThread::CurrentlyOn(BrowserThread::IO)) {
       BrowserThread::PostTask(
           BrowserThread::IO, FROM_HERE,
           base::Bind(&media_stream::MediaStreamDeviceSettings::PostResponse,
                      settings_, label, devices));
       return;
-    } else if (settings_) {
-      settings_->PostResponse(label, devices);
     }
+
+    settings_->PostResponse(label, devices);
   }
 
  private: