Remove media::VideoFrame from media::VideoCaptureDevice::Client interface

media/video/capture/video_capture_device_unittest.cc

Index: media/video/capture/video_capture_device_unittest.cc
diff --git a/media/video/capture/video_capture_device_unittest.cc b/media/video/capture/video_capture_device_unittest.cc
index f9d0ab2cb8abe84fbf7f06419b973d628b1e6c98..a9ec50ec8b8d6db75a5bf89e48094689ec611f2d 100644
--- a/media/video/capture/video_capture_device_unittest.cc
+++ b/media/video/capture/video_capture_device_unittest.cc
@@ -64,8 +64,10 @@ namespace media {
 
 class MockClient : public media::VideoCaptureDevice::Client {
  public:
-  MOCK_METHOD1(ReserveOutputBuffer,
-      scoped_refptr<media::VideoFrame>(const gfx::Size&));
+  MOCK_METHOD1(
+      ReserveOutputBuffer,
+      scoped_refptr<media::VideoFrame>(media::VideoFrame::Format format,
+                                       const gfx::Size&));
   MOCK_METHOD0(OnErr, void());
   MOCK_METHOD1(OnFrameInfo, void(const VideoCaptureCapability&));
   MOCK_METHOD1(OnFrameInfoChanged, void(const VideoCaptureCapability&));
@@ -89,9 +91,9 @@ class MockClient : public media::VideoCaptureDevice::Client {
     main_thread_->PostTask(FROM_HERE, frame_cb_);
   }
 
-  virtual void OnIncomingCapturedVideoFrame(
-      const scoped_refptr<media::VideoFrame>& frame,
-      base::Time timestamp) OVERRIDE {
+  virtual void OnIncomingCapturedI420Buffer(scoped_ptr<Buffer> buffer,

[ncarter (slow), 2013/10/29 18:42:17]

This method doesn't seem to match anything in VideoCaptureDevice::Client
interface. Does this compile?

[sheu, 2013/11/05 20:02:10]

Apparently not :-P

I didn't realize I had to build/run media_unittest as well.  All I'd been
running were browser_tests and content_unittest.

+                                            const gfx::Size& dimensions,
+                                            base::Time timestamp) OVERRIDE {
     main_thread_->PostTask(FROM_HERE, frame_cb_);
   }