Reconnect support for DirectShow video capture devices in parallel to MediaFoundation ones.

media/video/capture/video_capture_device.h

 // Copyright (c) 2012 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.
 //
 // VideoCaptureDevice is the abstract base class for realizing video capture
 // device support in Chromium. It provides the interface for OS dependent
 // implementations.
 // The class is created and functions are invoked on a thread owned by
 // VideoCaptureManager. Capturing is done on other threads, depending on the OS
 // specific implementation.
@@ skipping 13 matching lines @@
 class MEDIA_EXPORT VideoCaptureDevice {
  public:
 
   struct Name {
     // Friendly name of a device
     std::string device_name;
 
     // Unique name of a device. Even if there are multiple devices with the same
     // friendly name connected to the computer this will be unique.
     std::string unique_id;
+
+    friend bool operator==(const Name& left, const Name& right) {

[tommi (sloooow) - chröme, 2013/06/24 11:09:27]

In general operator overloading is discouraged but I'm assuming this is needed
for some STL feature?  Can you add comments that explains why we don't implement
an IsEqual/Equal method instead (which is what the style guide suggests).

Btw, why do you need the friend qualifier?

[mcasas, 2013/06/25 07:04:36]

Ok, supposition correct, done.
Actually no, I just put it out of routine.

+      return (left.unique_id == right.unique_id);
+    }
+    bool operator<(const Name& other) {
+      return (this->unique_id < other.unique_id);

[tommi (sloooow) - chröme, 2013/06/24 11:09:27]

remove "this->"

[mcasas, 2013/06/25 07:04:36]

Done.

+    }
   };
   typedef std::list<Name> Names;
 
   class MEDIA_EXPORT EventHandler {
    public:
 
     // Reserve an output buffer into which a video frame can be captured
     // directly. If all buffers are currently busy, returns NULL.
     //
     // The returned VideoFrames will always be allocated with a YV12 format. The
@@ skipping 89 matching lines @@
   // not permitted to make any additional calls to its EventHandler.
   virtual void DeAllocate() = 0;
 
   // Get the name of the capture device.
   virtual const Name& device_name() = 0;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_VIDEO_CAPTURE_VIDEO_CAPTURE_DEVICE_H_