Revert of Reenable media-device enumeration integration test.

chrome/browser/media/webrtc/webrtc_getmediadevices_browsertest.cc

Index: chrome/browser/media/webrtc/webrtc_getmediadevices_browsertest.cc
diff --git a/chrome/browser/media/webrtc/webrtc_getmediadevices_browsertest.cc b/chrome/browser/media/webrtc/webrtc_getmediadevices_browsertest.cc
index 48492e6b00f0a4f630399c6b7874942567dc98b2..6f10b78337918ab2a301a6e103bf6c8fb96f88aa 100644
--- a/chrome/browser/media/webrtc/webrtc_getmediadevices_browsertest.cc
+++ b/chrome/browser/media/webrtc/webrtc_getmediadevices_browsertest.cc
@@ -15,7 +15,6 @@
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/test/browser_test_utils.h"
-#include "media/audio/audio_device_description.h"
 #include "media/audio/audio_manager.h"
 #include "media/base/media_switches.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
@@ -28,9 +27,12 @@
 const char kDeviceKindVideoInput[] = "videoinput";
 const char kDeviceKindAudioOutput[] = "audiooutput";
 
+const char kSourceKindAudioInput[] = "audio";
+const char kSourceKindVideoInput[] = "video";
+
 }  // namespace
 
-// Integration test for WebRTC enumerateDevices. It always uses fake devices.
+// Integration test for WebRTC getMediaDevices. It always uses fake devices.
 // It needs to be a browser test (and not content browser test) to be able to
 // test that labels are cleared or not depending on if access to devices has
 // been granted.
@@ -63,9 +65,27 @@
     std::string group_id;
   };
 
-  void EnumerateDevices(content::WebContents* tab,
-                        std::vector<MediaDeviceInfo>* devices) {
-    std::string devices_as_json = ExecuteJavascript("enumerateDevices()", tab);
+  bool HasOutputDevices() {
+    // There's no fake audio output devices supported yet. We can't test audio
+    // output devices on bots with no output devices, so skip testing for that
+    // on such bots. We cache the result since querying for devices can take
+    // considerable time.
+    if (!has_audio_output_devices_initialized_) {
+      has_audio_output_devices_ =
+          media::AudioManager::Get()->HasAudioOutputDevices();
+      has_audio_output_devices_initialized_ = true;
+    }
+    return has_audio_output_devices_;
+  }
+
+  // If |get_sources| is true, use getSources API and leave groupId empty,
+  // otherwise use getMediaDevices API.
+  void GetMediaDevicesOrSources(content::WebContents* tab,
+                                std::vector<MediaDeviceInfo>* devices,
+                                bool get_sources) {
+    std::string devices_as_json =
+        ExecuteJavascript(get_sources ? "getSources()" : "getMediaDevices()",
+                          tab);
     EXPECT_FALSE(devices_as_json.empty());
 
     int error_code;
@@ -89,34 +109,41 @@
       const base::DictionaryValue* dict;
       MediaDeviceInfo device;
       ASSERT_TRUE((*it)->GetAsDictionary(&dict));
-      ASSERT_TRUE(dict->GetString("deviceId", &device.device_id));
+      ASSERT_TRUE(dict->GetString(get_sources ? "id" : "deviceId",
+                                  &device.device_id));
       ASSERT_TRUE(dict->GetString("kind", &device.kind));
       ASSERT_TRUE(dict->GetString("label", &device.label));
-      ASSERT_TRUE(dict->GetString("groupId", &device.group_id));
+      if (!get_sources)
+        ASSERT_TRUE(dict->GetString("groupId", &device.group_id));
 
       // Should be HMAC SHA256.
-      if (!media::AudioDeviceDescription::IsDefaultDevice(device.device_id) &&
-          !(device.device_id ==
-            media::AudioDeviceDescription::kCommunicationsDeviceId)) {
-        EXPECT_EQ(64ul, device.device_id.length());
-        EXPECT_TRUE(
-            base::ContainsOnlyChars(device.device_id, "0123456789abcdef"));
+      EXPECT_EQ(64ul, device.device_id.length());
+      EXPECT_TRUE(base::ContainsOnlyChars(device.device_id,
+                                          "0123456789abcdef"));
+
+      const char* kAudioInputKind =
+          get_sources ? kSourceKindAudioInput : kDeviceKindAudioInput;
+      const char* kVideoInputKind =
+          get_sources ? kSourceKindVideoInput : kDeviceKindVideoInput;
+      if (get_sources) {
+        EXPECT_TRUE(device.kind == kAudioInputKind ||
+                    device.kind == kVideoInputKind);
+      } else {
+        EXPECT_TRUE(device.kind == kAudioInputKind ||
+                    device.kind == kVideoInputKind ||
+                    device.kind == kDeviceKindAudioOutput);
       }
-
-      EXPECT_TRUE(device.kind == kDeviceKindAudioInput ||
-                  device.kind == kDeviceKindVideoInput ||
-                  device.kind == kDeviceKindAudioOutput);
-      if (device.kind == kDeviceKindAudioInput) {
+      if (device.kind == kAudioInputKind) {
         found_audio_input = true;
-      } else if (device.kind == kDeviceKindVideoInput) {
+      } else if (device.kind == kVideoInputKind) {
         found_video_input = true;
       } else {
         found_audio_output = true;
       }
 
-      // enumerateDevices doesn't have group ID support for video input devices.
-      // TODO(guidou): remove this once http://crbug.com/627793 is fixed.
-      if (device.kind == kDeviceKindVideoInput) {
+      // getSources doesn't have group ID support. getMediaDevices doesn't have
+      // group ID support for video input devices.
+      if (get_sources || device.kind == kDeviceKindVideoInput) {
         EXPECT_TRUE(device.group_id.empty());
       } else {
         EXPECT_FALSE(device.group_id.empty());
@@ -127,6 +154,21 @@
 
     EXPECT_TRUE(found_audio_input);
     EXPECT_TRUE(found_video_input);
+    if (get_sources) {
+      EXPECT_FALSE(found_audio_output);
+    } else {
+      EXPECT_EQ(HasOutputDevices(), found_audio_output);
+    }
+  }
+
+  void GetMediaDevices(content::WebContents* tab,
+                     std::vector<MediaDeviceInfo>* devices) {
+    GetMediaDevicesOrSources(tab, devices, false);
+  }
+
+  void GetSources(content::WebContents* tab,
+                  std::vector<MediaDeviceInfo>* sources) {
+    GetMediaDevicesOrSources(tab, sources, true);
   }
 
   bool has_audio_output_devices_initialized_;
@@ -138,16 +180,18 @@
                         WebRtcGetMediaDevicesBrowserTest,
                         testing::ValuesIn(kParamsToRunTestsWith));
 
-IN_PROC_BROWSER_TEST_P(WebRtcGetMediaDevicesBrowserTest,
-                       EnumerateDevicesWithoutAccess) {
-  ASSERT_TRUE(embedded_test_server()->Start());
-  GURL url(embedded_test_server()->GetURL(kMainWebrtcTestHtmlPage));
-  ui_test_utils::NavigateToURL(browser(), url);
-  content::WebContents* tab =
-      browser()->tab_strip_model()->GetActiveWebContents();
-
-  std::vector<MediaDeviceInfo> devices;
-  EnumerateDevices(tab, &devices);
+// getMediaDevices has been removed and will be replaced
+// MediaDevices.enumerateDevices. http://crbug.com/388648.
+IN_PROC_BROWSER_TEST_P(WebRtcGetMediaDevicesBrowserTest,
+                       DISABLED_GetMediaDevicesWithoutAccess) {
+  ASSERT_TRUE(embedded_test_server()->Start());
+  GURL url(embedded_test_server()->GetURL(kMainWebrtcTestHtmlPage));
+  ui_test_utils::NavigateToURL(browser(), url);
+  content::WebContents* tab =
+      browser()->tab_strip_model()->GetActiveWebContents();
+
+  std::vector<MediaDeviceInfo> devices;
+  GetMediaDevices(tab, &devices);
 
   // Labels should be empty if access has not been allowed.
   for (std::vector<MediaDeviceInfo>::iterator it = devices.begin();
@@ -156,8 +200,11 @@
   }
 }
 
-IN_PROC_BROWSER_TEST_P(WebRtcGetMediaDevicesBrowserTest,
-                       EnumerateDevicesWithAccess) {
+// getMediaDevices has been removed and will be replaced
+// MediaDevices.enumerateDevices. http://crbug.com/388648.
+// Disabled, fails due to http://crbug.com/382391.
+IN_PROC_BROWSER_TEST_P(WebRtcGetMediaDevicesBrowserTest,
+                       DISABLED_GetMediaDevicesWithAccess) {
   ASSERT_TRUE(embedded_test_server()->Start());
   GURL url(embedded_test_server()->GetURL(kMainWebrtcTestHtmlPage));
   ui_test_utils::NavigateToURL(browser(), url);
@@ -167,7 +214,7 @@
   EXPECT_TRUE(GetUserMediaAndAccept(tab));
 
   std::vector<MediaDeviceInfo> devices;
-  EnumerateDevices(tab, &devices);
+  GetMediaDevices(tab, &devices);
 
   // Labels should be non-empty if access has been allowed.
   for (std::vector<MediaDeviceInfo>::iterator it = devices.begin();
@@ -175,3 +222,74 @@
     EXPECT_TRUE(!it->label.empty());
   }
 }
+
+// getMediaDevices has been removed and will be replaced
+// MediaDevices.enumerateDevices. http://crbug.com/388648.
+IN_PROC_BROWSER_TEST_P(WebRtcGetMediaDevicesBrowserTest,
+                       DISABLED_GetMediaDevicesEqualsGetSourcesWithoutAccess) {
+  ASSERT_TRUE(embedded_test_server()->Start());
+  GURL url(embedded_test_server()->GetURL(kMainWebrtcTestHtmlPage));
+  ui_test_utils::NavigateToURL(browser(), url);
+  content::WebContents* tab =
+      browser()->tab_strip_model()->GetActiveWebContents();
+
+  std::vector<MediaDeviceInfo> devices;
+  GetMediaDevices(tab, &devices);
+
+  std::vector<MediaDeviceInfo> sources;
+  GetSources(tab, &sources);
+
+  std::vector<MediaDeviceInfo>::iterator sources_it = sources.begin();
+  for (std::vector<MediaDeviceInfo>::iterator devices_it = devices.begin();
+       devices_it != devices.end(); ++devices_it) {
+    if (devices_it->kind == kDeviceKindAudioOutput)
+      continue;
+    EXPECT_STREQ(devices_it->device_id.c_str(), sources_it->device_id.c_str());
+    if (devices_it->kind == kDeviceKindAudioInput) {
+      EXPECT_STREQ(kSourceKindAudioInput, sources_it->kind.c_str());
+    } else {
+      EXPECT_STREQ(kSourceKindVideoInput, sources_it->kind.c_str());
+    }
+    EXPECT_TRUE(devices_it->label.empty());
+    EXPECT_TRUE(sources_it->label.empty());
+    ++sources_it;
+  }
+  EXPECT_EQ(sources.end(), sources_it);
+}
+
+// getMediaDevices has been removed and will be replaced
+// MediaDevices.enumerateDevices. http://crbug.com/388648.
+// Disabled, fails due to http://crbug.com/382391.
+IN_PROC_BROWSER_TEST_P(WebRtcGetMediaDevicesBrowserTest,
+                       DISABLED_GetMediaDevicesEqualsGetSourcesWithAccess) {
+  ASSERT_TRUE(embedded_test_server()->Start());
+  GURL url(embedded_test_server()->GetURL(kMainWebrtcTestHtmlPage));
+  ui_test_utils::NavigateToURL(browser(), url);
+  content::WebContents* tab =
+      browser()->tab_strip_model()->GetActiveWebContents();
+
+  EXPECT_TRUE(GetUserMediaAndAccept(tab));
+
+  std::vector<MediaDeviceInfo> devices;
+  GetMediaDevices(tab, &devices);
+
+  std::vector<MediaDeviceInfo> sources;
+  GetSources(tab, &sources);
+
+  std::vector<MediaDeviceInfo>::iterator sources_it = sources.begin();
+  for (std::vector<MediaDeviceInfo>::iterator devices_it = devices.begin();
+       devices_it != devices.end(); ++devices_it) {
+    if (devices_it->kind == kDeviceKindAudioOutput)
+      continue;
+    EXPECT_STREQ(devices_it->device_id.c_str(), sources_it->device_id.c_str());
+    if (devices_it->kind == kDeviceKindAudioInput) {
+      EXPECT_STREQ(kSourceKindAudioInput, sources_it->kind.c_str());
+    } else {
+      EXPECT_STREQ(kSourceKindVideoInput, sources_it->kind.c_str());
+    }
+    EXPECT_TRUE(!devices_it->label.empty());
+    EXPECT_STREQ(devices_it->label.c_str(), sources_it->label.c_str());
+    ++sources_it;
+  }
+  EXPECT_EQ(sources.end(), sources_it);
+}