Change class VideoCaptureDevice::Name to struct VideoCaptureDeviceDescriptor

content/browser/media/media_internals_unittest.cc

Index: content/browser/media/media_internals_unittest.cc
diff --git a/content/browser/media/media_internals_unittest.cc b/content/browser/media/media_internals_unittest.cc
index b8484d22e21b96037ac823714b018165b52d9e5b..b18463e21f14511afb3f66dc71b88a8731c58adf 100644
--- a/content/browser/media/media_internals_unittest.cc
+++ b/content/browser/media/media_internals_unittest.cc
@@ -112,33 +112,31 @@ class MediaInternalsVideoCaptureDeviceTest : public testing::Test,
   MediaInternals::UpdateCallback update_cb_;
 };
 
+// TODO(chfremer): Consider removing this. This test seem be
+// a duplicate implementation of the functionality under test.
+// https://crbug.com/630694
 #if defined(OS_WIN) || defined(OS_MACOSX) || defined(OS_LINUX) || \
     defined(OS_ANDROID)
 TEST_F(MediaInternalsVideoCaptureDeviceTest,
        AllCaptureApiTypesHaveProperStringRepresentation) {
-  typedef media::VideoCaptureDevice::Name VideoCaptureDeviceName;
-  typedef std::map<VideoCaptureDeviceName::CaptureApiType, std::string>
-      CaptureApiTypeStringMap;
+  using VideoCaptureApi = media::VideoCaptureApi;
+  typedef std::map<VideoCaptureApi, std::string> CaptureApiTypeStringMap;
   CaptureApiTypeStringMap m;
-#if defined(OS_LINUX)
-  m[VideoCaptureDeviceName::V4L2_SINGLE_PLANE] = "V4L2 SPLANE";
-#elif defined(OS_WIN)
-  m[VideoCaptureDeviceName::MEDIA_FOUNDATION] = "Media Foundation";
-  m[VideoCaptureDeviceName::DIRECT_SHOW] = "Direct Show";
-#elif defined(OS_MACOSX)
-  m[VideoCaptureDeviceName::AVFOUNDATION] = "AV Foundation";
-  m[VideoCaptureDeviceName::DECKLINK] = "DeckLink";
-#elif defined(OS_ANDROID)
-  m[VideoCaptureDeviceName::API1] = "Camera API1";
-  m[VideoCaptureDeviceName::API2_LEGACY] = "Camera API2 Legacy";
-  m[VideoCaptureDeviceName::API2_FULL] = "Camera API2 Full";
-  m[VideoCaptureDeviceName::API2_LIMITED] = "Camera API2 Limited";
-  m[VideoCaptureDeviceName::TANGO] = "Tango API";
-#endif
-  EXPECT_EQ(media::VideoCaptureDevice::Name::API_TYPE_UNKNOWN, m.size());
+  m[VideoCaptureApi::LINUX_V4L2_SINGLE_PLANE] = "V4L2 SPLANE";
+  m[VideoCaptureApi::WINDOWS_MEDIA_FOUNDATION] = "Media Foundation";
+  m[VideoCaptureApi::WINDOWS_DIRECT_SHOW] = "Direct Show";
+  m[VideoCaptureApi::MACOSX_AVFOUNDATION] = "AV Foundation";
+  m[VideoCaptureApi::MACOSX_DECKLINK] = "DeckLink";
+  m[VideoCaptureApi::ANDROID_API1] = "Camera API1";
+  m[VideoCaptureApi::ANDROID_API2_LEGACY] = "Camera API2 Legacy";
+  m[VideoCaptureApi::ANDROID_API2_FULL] = "Camera API2 Full";
+  m[VideoCaptureApi::ANDROID_API2_LIMITED] = "Camera API2 Limited";
+  m[VideoCaptureApi::ANDROID_TANGO] = "Tango API";
+  EXPECT_EQ(static_cast<size_t>(VideoCaptureApi::UNKNOWN), m.size());
   for (CaptureApiTypeStringMap::iterator it = m.begin(); it != m.end(); ++it) {

[mcasas, 2016/07/30 00:06:11]

nit: for (const auto& capture_api : m) ...

[chfremer, 2016/08/01 18:53:12]

Done.

-    const VideoCaptureDeviceName device_name("dummy", "dummy", it->first);
-    EXPECT_EQ(it->second, device_name.GetCaptureApiTypeString());
+    media::VideoCaptureDeviceDescriptor descriptor;
+    descriptor.capture_api = it->first;
+    EXPECT_EQ(it->second, descriptor.GetCaptureApiTypeString());

[mcasas, 2016/07/30 00:06:11]

EXPECT_STREQ ...?

[chfremer, 2016/08/01 18:53:12]

Done.

   }
 }
 #endif
@@ -177,25 +175,25 @@ TEST_F(MediaInternalsVideoCaptureDeviceTest,
       kFrameRate, kPixelFormat);
   media::VideoCaptureFormats formats{};
   formats.push_back(format_hd);
-  const media::VideoCaptureDeviceInfo device_info(
+  VideoCaptureDeviceInfo device_info;
+  device_info.supported_formats = formats;
+  device_info.descriptor.device_id = "dummy";
+  device_info.descriptor.display_name = "dummy";
 #if defined(OS_MACOSX)
-      media::VideoCaptureDevice::Name(
-          "dummy", "dummy", media::VideoCaptureDevice::Name::AVFOUNDATION),
+  device_info.descriptor.capture_api =
+      media::VideoCaptureApi::MACOSX_AVFOUNDATION;
 #elif defined(OS_WIN)
-      media::VideoCaptureDevice::Name("dummy", "dummy",
-          media::VideoCaptureDevice::Name::DIRECT_SHOW),
+  device_info.descriptor.capture_api =
+      media::VideoCaptureApi::WINDOWS_DIRECT_SHOW;
 #elif defined(OS_LINUX)
-      media::VideoCaptureDevice::Name(
-          "dummy", "/dev/dummy",
-          media::VideoCaptureDevice::Name::V4L2_SINGLE_PLANE),
+  device_info.descriptor.device_id = "/dev/dummy";
+  device_info.descriptor.capture_api =
+      media::VideoCaptureApi::LINUX_V4L2_SINGLE_PLANE;
 #elif defined(OS_ANDROID)
-      media::VideoCaptureDevice::Name("dummy", "dummy",
-          media::VideoCaptureDevice::Name::API2_LEGACY),
-#else
-      media::VideoCaptureDevice::Name("dummy", "dummy"),
+  device_info.descriptor.capture_api =
+      media::VideoCaptureApi::ANDROID_API2_LEGACY;
 #endif
-      formats);
-  media::VideoCaptureDeviceInfos device_infos{};
+  VideoCaptureDeviceInfos device_infos{};
   device_infos.push_back(device_info);
 
   // When updating video capture capabilities, the update will serialize