Changed thread_checker on VideoCaptureImpl and VideoCaptureImplManager.

content/renderer/media/video_capture_impl.h

Index: content/renderer/media/video_capture_impl.h
diff --git a/content/renderer/media/video_capture_impl.h b/content/renderer/media/video_capture_impl.h
index d7d6eb0791ed0364cac02bda567e4b4321dbbf1f..21fa91fc305af27621372bceecd472510bb382c3 100644
--- a/content/renderer/media/video_capture_impl.h
+++ b/content/renderer/media/video_capture_impl.h
@@ -45,6 +45,10 @@ class VideoFrame;
 
 namespace content {
 
+// VideoCaptureImpl is used as a representation of capture device in renderer
+// process. It provides communication to its clients about the state of the
+// capture device.
+// All methods must be called on the IO thread.

[mcasas, 2015/03/06 19:18:29]

Comments for the class are at the file level. Owners might
say otherwise but I'd put those next to the class; 

[emircan, 2015/03/10 23:38:36]

I moved the existing comments to here instead.

 class CONTENT_EXPORT VideoCaptureImpl
     : public VideoCaptureMessageFilter::Delegate {
  public:
@@ -173,8 +177,9 @@ class CONTENT_EXPORT VideoCaptureImpl
   bool suspended_;
   VideoCaptureState state_;
 
-  // |weak_factory_| and |thread_checker_| are bound to the IO thread.
-  base::ThreadChecker thread_checker_;
+  // Hold a pointer to the IO message loop to check we operate on the right
+  // thread.
+  scoped_refptr<base::MessageLoopProxy> io_message_loop_;

[mcasas, 2015/03/06 19:18:29]

Make const.

[emircan, 2015/03/10 23:38:36]

This may be overwritten inside Init() method, see there for more info.

 
   // WeakPtrFactory pointing back to |this| object, for use with
   // media::VideoFrames constructed in OnBufferReceived() from buffers cached