Refactor how the DeviceMonitor is started.

content/browser/media/media_devices_monitor.cc

Index: content/browser/media/media_devices_monitor.cc
diff --git a/content/browser/media/media_devices_monitor.cc b/content/browser/media/media_devices_monitor.cc
index 8f239a9b10aa4f03e39f0aa72f7e4b21335c0024..50cc7511a06584a968a367f7fcfa792e6a185864 100644
--- a/content/browser/media/media_devices_monitor.cc
+++ b/content/browser/media/media_devices_monitor.cc
@@ -12,10 +12,7 @@ namespace content {
 namespace {
 void EnsureMonitorCaptureDevicesInternal(
     MediaStreamManager* media_stream_manager) {
-  // TODO(perkj): The argument list suggests that this is the wrong design. Can
-  // we add a separate method for device monitoring?
-  media_stream_manager->EnumerateDevices(
-      NULL, -1, -1, NULL, -1, MEDIA_DEVICE_AUDIO_CAPTURE, GURL());
+  media_stream_manager->StartMonitoring();

[no longer working on chromium, 2013/11/04 15:51:22]

This EnsureMonitorCaptureDevicesInternal is triggered event time there is a
device change event, the naming of StartMonitoring() indicates that it is wrong
to start monitoring multiple times.
Can you just add a interface called EnsureMonitoring() to the MSM and move
Startmonitoring() back to its private method?

 }
 }