enable video capture to support sharing across multiple renderer processes

content/browser/renderer_host/media/video_capture_controller_event_handler.h

 // Copyright (c) 2011 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.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_CONTROLLER_EVENT_HANDLER_H_
 #define CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_CONTROLLER_EVENT_HANDLER_H_
 
 #include "base/shared_memory.h"
 #include "base/time.h"
 #include "content/common/content_export.h"
 
 // ID used for identifying an object of VideoCaptureController.
 struct CONTENT_EXPORT VideoCaptureControllerID {

[scherkus (not reviewing), 2011/10/19 18:02:21]

I forget the history of this, but why is this not a typedef of an int?

[wjia(left Chromium), 2011/10/21 00:56:13]

Originally, there was a routing_id here. Since message is changed from routed to
control, the routing_id was gone. But I still keep it as struct in case we need
add more info here. If you are strongly against it, I can use typedef.

-  VideoCaptureControllerID(int device_id);
+  explicit VideoCaptureControllerID(int device_id);
 
   bool operator<(const VideoCaptureControllerID& vc) const;
+  bool operator==(const VideoCaptureControllerID& vc) const;
 
   int device_id;
 };
 
 // VideoCaptureControllerEventHandler is the interface for
 // VideoCaptureController to notify clients about the events such as
 // BufferReady, FrameInfo, Error, etc.
 class CONTENT_EXPORT VideoCaptureControllerEventHandler {
  public:
   // An Error has occurred in the VideoCaptureDevice.
@@ skipping 16 matching lines @@
                            int frame_rate) = 0;
 
   // Report that this object can be deleted.
   virtual void OnReadyToDelete(const VideoCaptureControllerID& id) = 0;
 
  protected:
   virtual ~VideoCaptureControllerEventHandler() {}
 };
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_CONTROLLER_EVENT_HANDLER_H_