Follow up patch on issue 7192007, containing mainly style fixes.

content/browser/renderer_host/media/video_capture_manager.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.
 
 // VideoCaptureManager is used to open/close, start/stop as well as enumerate
 // available video capture devices. All functions are expected to be called from
 // the Browser::IO thread. VideoCaptureManager will open OS dependent instances
 // of VideoCaptureDevice. A device can only be opened once.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_MANAGER_H_
@@ skipping 11 matching lines @@
 namespace media_stream {
 
 // VideoCaptureManager opens/closes and start/stops video capture devices.
 class VideoCaptureManager : public MediaStreamProvider {
  public:
   // Calling |Start| of this id will open the first device, even though open has
   // not been called. This is used to be able to use video capture devices
   // before MediaStream is implemented in Chrome and WebKit.
   enum { kStartOpenSessionId = 1 };
 
-  // Called to get a pointer to the singleton
+  // Called to get a pointer to the singleton.
   static VideoCaptureManager* Get();
 
   // Implements MediaStreamProvider.
   virtual void Register(MediaStreamProviderListener* listener);
 
   virtual void Unregister();
 
   virtual void EnumerateDevices();
 
   virtual int Open(const StreamDeviceInfo& device);
 
   virtual void Close(int capture_session_id);
 
   // Functions used to start and stop media flow.
   // Start allocates the device and no other application can use the device
   // before Stop is called. Captured video frames will be delivered to
   // video_capture_receiver.
   void Start(const media::VideoCaptureParams& capture_params,
              media::VideoCaptureDevice::EventHandler* video_capture_receiver);
 
   // Stops capture device referenced by |capture_session_id|. No more frames
   // will be delivered to the frame receiver, and |stopped_task| will be called.
-  void Stop(const media::VideoCaptureSessionId capture_session_id,
+  void Stop(const media::VideoCaptureSessionId& capture_session_id,
             Task* stopped_task);
 
   // A capture device error has occurred for |capture_session_id|. The device
   // won't stream any more captured frames.
   void Error(const media::VideoCaptureSessionId& capture_session_id);
 
   virtual ~VideoCaptureManager();
 
   // Used by unit test to make sure a fake device is used instead of a real
   // video capture device. Due to timing requirements, the function must be
@@ skipping 27 matching lines @@
   void PostOnClosed(int capture_session_id);
   void PostOnDevicesEnumerated(const StreamDeviceInfoArray& devices);
   void PostOnError(int capture_session_id, MediaStreamProviderError error);
 
   // Helpers
   void GetAvailableDevices(media::VideoCaptureDevice::Names* device_names);
   bool DeviceOpened(const media::VideoCaptureDevice::Name& device_name);
   bool DeviceOpened(const StreamDeviceInfo& device_info);
   bool IsOnCaptureDeviceThread() const;
 
-  // Thread for all calls to VideoCaptureDevice
+  // Thread for all calls to VideoCaptureDevice.
   base::Thread vc_device_thread_;
 
-  // Only accessed on Browser::IO thread
+  // Only accessed on Browser::IO thread.
   MediaStreamProviderListener* listener_;
   int new_capture_session_id_;
 
-  // Only accessed from vc_device_thread_
+  // Only accessed from vc_device_thread_.
   // VideoCaptureManager owns all VideoCaptureDevices and is responsible for
   // deleting the instances when they are not used any longer.
   typedef std::map<int, media::VideoCaptureDevice*> VideoCaptureDevices;
   VideoCaptureDevices devices_;
 
   // Set to true if using fake devices for testing, false by default.
   bool use_fake_device_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoCaptureManager);
 };
 
 }  // namespace media_stream
 
 DISABLE_RUNNABLE_METHOD_REFCOUNT(media_stream::VideoCaptureManager);
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_MANAGER_H_