Changed thread_checker on VideoCaptureImpl and VideoCaptureImplManager.

content/renderer/media/video_capture_impl.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.
 
 // VideoCaptureImpl represents a capture device in renderer process. It provides
 // interfaces for clients to Start/Stop capture. It also communicates to clients
 // when buffer is ready, state of capture device is changed.
 
 // VideoCaptureImpl is also a delegate of VideoCaptureMessageFilter which relays
 // operation of a capture device to the browser process and receives responses
@@ skipping 27 matching lines @@
 namespace gpu {
 struct MailboxHolder;
 }  // namespace gpu
 
 namespace media {
 class VideoFrame;
 }  // namespace media
 
 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:
   ~VideoCaptureImpl() override;
 
   VideoCaptureImpl(media::VideoCaptureSessionId session_id,
                    VideoCaptureMessageFilter* filter);
 
   // Start listening to IPC messages.
   void Init();
@@ skipping 108 matching lines @@
   // Member params_ represents the video format requested by the
   // client to this class via StartCapture().
   media::VideoCaptureParams params_;
 
   // The device's first captured frame timestamp sent from browser process side.
   base::TimeTicks first_frame_timestamp_;
 
   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
   // in |client_buffers_|.
   // NOTE: Weak pointers must be invalidated before all other member variables.
   base::WeakPtrFactory<VideoCaptureImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoCaptureImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_MEDIA_VIDEO_CAPTURE_IMPL_H_