[Mojo Video Capture] Use string keys instead of VideoCaptureDeviceDescriptor

services/video_capture/public/interfaces/video_capture_device_factory.mojom

Index: services/video_capture/public/interfaces/video_capture_device_factory.mojom
diff --git a/services/video_capture/public/interfaces/video_capture_device_factory.mojom b/services/video_capture/public/interfaces/video_capture_device_factory.mojom
index 01752012226cdb183d43f83111506f992b9b1f57..bf0aa378590947489a609d5d10b7479ed028b8a7 100644
--- a/services/video_capture/public/interfaces/video_capture_device_factory.mojom
+++ b/services/video_capture/public/interfaces/video_capture_device_factory.mojom
@@ -17,21 +17,20 @@ enum DeviceAccessResultCode {
 
 // Enables access to a set of video capture devices.
 // Typical operation is to first call EnumerateDeviceDescriptors() to obtain
-// information about available devices. The obtained descriptors can then be
-// used to either obtain the supported formats for a device using
-// GetSupportedFormats(), or to create an instance of VideoCaptureDevice for
-// the device using CreateDevice().
-// TODO(chfremer): Consider using a simple string identifier instead of a
-// VideoCaptureDeviceDescriptor.
-// https://crbug.com/637439
+// information about available devices. The |device_id| of the descriptors can
+// subsequently be used to either obtain the supported formats for a device
+// using GetSupportedFormats(), or to create an instance of
+// VideoCaptureDeviceProxy using CreateDeviceProxy().
+// The factory guarantees that no two device descriptors it returns use the
+// same |device_id|.
 interface VideoCaptureDeviceFactory {
   EnumerateDeviceDescriptors()
       => (array<VideoCaptureDeviceDescriptor> descriptors);
 
-  GetSupportedFormats(VideoCaptureDeviceDescriptor device_descriptor)
+  GetSupportedFormats(string device_id)

[mcasas, 2016/11/14 19:02:02]

As a side comment for the future, we might like
to use UUIDs [1] here to allow for security by obfuscation,
now it's fine to use |device_id| (although technically
is an internal detail, e.g. /dev/video0 etc).

[1]
https://cs.chromium.org/chromium/src/base/guid.h?q=GenerateGUID&sq=package:ch...

[chfremer, 2016/11/14 19:23:29]

Acknowledged.

       => (array<VideoCaptureFormat> supported_formats);
 
-  // Provides exclusive access to the device identified by |device_descriptor|.
+  // Provides exclusive access to the device identified by |device_id|.
   // The access is valid until either the message pipe associated with
   // |proxy_request| is closed by the client, or a subsequent call to
   // CreateDeviceProxy() is made. When a subsequent call is made while the
@@ -42,7 +41,7 @@ interface VideoCaptureDeviceFactory {
   // |proxy_request| has been (asynchronously) discarded a subsequent call to
   // CreateDeviceProxy() would succeed if it were to deny access while still
   // in use.
-  CreateDeviceProxy(VideoCaptureDeviceDescriptor device_descriptor,
+  CreateDeviceProxy(string device_id,
                     VideoCaptureDeviceProxy& proxy_request)
       => (DeviceAccessResultCode result_code);
 };