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

content/browser/media_devices_monitor.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/public/browser/media_devices_monitor.h"
 
 #include "content/browser/browser_main_loop.h"
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 #include "content/public/browser/browser_thread.h"
 
 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