Sort camera device list to use front camera first

media/capture/video/video_capture_device_descriptor.cc

 // Copyright 2016 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 "media/capture/video/video_capture_device_descriptor.h"
 
+#include <map>
+
 #include "base/logging.h"
 
 namespace media {
 
 VideoCaptureDeviceDescriptor::VideoCaptureDeviceDescriptor()
     : facing(VideoFacingMode::MEDIA_VIDEO_FACING_NONE),
       capture_api(VideoCaptureApi::UNKNOWN),
       transport_type(VideoCaptureTransportType::OTHER_TRANSPORT) {}
 
 VideoCaptureDeviceDescriptor::VideoCaptureDeviceDescriptor(
@@ skipping 19 matching lines @@
       model_id(model_id),
       facing(facing),
       capture_api(capture_api),
       transport_type(transport_type) {}
 
 VideoCaptureDeviceDescriptor::~VideoCaptureDeviceDescriptor() {}
 
 VideoCaptureDeviceDescriptor::VideoCaptureDeviceDescriptor(
     const VideoCaptureDeviceDescriptor& other) = default;
 
+bool VideoCaptureDeviceDescriptor::operator<(
+    const VideoCaptureDeviceDescriptor& other) const {
+  std::map<media::VideoFacingMode, size_t> facing_mapping{

[Guido Urdaneta, 2017/01/24 10:42:07]

nit: Optional. the map is fine, but maybe you can also use an array:
int facing_mapping[NUM_MEDIA_VIDEO_FACING_MODE];
facing_mapping[MEDIA_VIDEO_FACING_USER] = 0;
facing_mapping[MEDIA_VIDEO_FACING_ENVIRONMENT] = 1;
facing_mapping[MEDIA_VIDEO_FACING_NONE] = 2;

[henryhsu, 2017/01/25 03:28:18]

Done.

+      {MEDIA_VIDEO_FACING_USER, 0},
+      {MEDIA_VIDEO_FACING_ENVIRONMENT, 1},
+      {MEDIA_VIDEO_FACING_NONE, 2}};
+  if (facing_mapping.at(facing) < facing_mapping.at(other.facing))
+    return true;
+  if (device_id < other.device_id)
+    return true;
+  return capture_api < other.capture_api;
+}
+
 const char* VideoCaptureDeviceDescriptor::GetCaptureApiTypeString() const {
   switch (capture_api) {
     case VideoCaptureApi::LINUX_V4L2_SINGLE_PLANE:
       return "V4L2 SPLANE";
     case VideoCaptureApi::WIN_MEDIA_FOUNDATION:
       return "Media Foundation";
     case VideoCaptureApi::WIN_DIRECT_SHOW:
       return "Direct Show";
     case VideoCaptureApi::MACOSX_AVFOUNDATION:
       return "AV Foundation";
@@ skipping 16 matching lines @@
   }
 }
 
 std::string VideoCaptureDeviceDescriptor::GetNameAndModel() const {
   if (model_id.empty())
     return display_name;
   return display_name + " (" + model_id + ")";
 }
 
 }  // namespace media