Minor cleanup to remove the static thread-safe methods on BrowserMainLoop. Initially I thought this…

content/browser/media_devices_monitor.cc

Index: content/browser/media_devices_monitor.cc
===================================================================
--- content/browser/media_devices_monitor.cc	(revision 207505)
+++ content/browser/media_devices_monitor.cc	(working copy)
@@ -11,10 +11,17 @@
 namespace content {
 
 void EnsureMonitorCaptureDevices() {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+
   // Post a EnumerateDevices() API to MSM to start the monitoring.
-  BrowserMainLoop::GetMediaStreamManager()->EnumerateDevices(
-      NULL, -1, -1, MEDIA_DEVICE_AUDIO_CAPTURE, GURL());
+  MediaStreamRequester* requester = NULL;
+  BrowserThread::PostTask(
+      BrowserThread::IO,
+      FROM_HERE,
+      base::Bind(base::IgnoreResult(&MediaStreamManager::EnumerateDevices),

[scherkus (not reviewing), 2013/06/21 16:18:42]

oh gross :(

yeah ... let's go back to the old method

sorry about that!

+                 base::Unretained(
+                     BrowserMainLoop::GetInstance()->media_stream_manager()),
+                 requester, -1, -1, MEDIA_DEVICE_AUDIO_CAPTURE, GURL()));
 }
 
 }  // namespace content