Hook up EncodedVideoSource on the browser side

content/browser/renderer_host/media/web_contents_video_capture_device_unittest.cc

 // 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.
 
 #include "content/browser/renderer_host/media/web_contents_video_capture_device.h"
 
 #include "base/bind_helpers.h"
 #include "base/debug/debugger.h"
 #include "base/run_loop.h"
 #include "base/test/test_timeouts.h"
 #include "base/time.h"
 #include "base/timer.h"
 #include "content/browser/browser_thread_impl.h"
 #include "content/browser/renderer_host/media/video_capture_buffer_pool.h"
 #include "content/browser/renderer_host/media/video_capture_controller.h"
 #include "content/browser/renderer_host/media/video_capture_oracle.h"
 #include "content/browser/renderer_host/media/web_contents_capture_util.h"
 #include "content/browser/renderer_host/render_view_host_factory.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 #include "content/browser/renderer_host/test_render_view_host.h"
 #include "content/port/browser/render_widget_host_view_frame_subscriber.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/test/mock_render_process_host.h"
 #include "content/public/test/test_browser_context.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "content/public/test/test_utils.h"
 #include "content/test/test_web_contents.h"
+#include "media/base/encoded_bitstream_buffer.h"
 #include "media/base/video_util.h"
 #include "media/base/yuv_convert.h"
 #include "media/video/capture/video_capture_types.h"
 #include "skia/ext/platform_canvas.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/skia/include/core/SkColor.h"
 
 namespace content {
 namespace {
 
@@ skipping 310 matching lines @@
       base::AutoLock guard(lock_);
       ASSERT_TRUE(error_encountered_);
     }
   }
 
   bool HasError() {
     base::AutoLock guard(lock_);
     return error_encountered_;
   }
 
-  virtual scoped_refptr<media::VideoFrame> ReserveOutputBuffer() OVERRIDE {
+  virtual scoped_refptr<media::VideoFrame> ReserveOutputVideoFrame() OVERRIDE {
     return VideoCaptureController::ReserveI420VideoFrame(
         buffer_pool_,
         gfx::Size(kTestWidth, kTestHeight));
   }
 
+  virtual scoped_refptr<media::EncodedBitstreamBuffer>
+  ReserveOutputEncodedBitstreamBuffer() OVERRIDE {
+    return NULL;
+  }
+
   virtual void OnIncomingCapturedFrame(
       const uint8* data,
       int length,
       base::Time timestamp,
       int rotation,
       bool flip_vert,
       bool flip_horiz) OVERRIDE {
     FAIL();
   }
 
   virtual void OnIncomingCapturedVideoFrame(
       const scoped_refptr<media::VideoFrame>& frame,
       base::Time timestamp) OVERRIDE {
     EXPECT_EQ(gfx::Size(kTestWidth, kTestHeight), frame->coded_size());
     EXPECT_EQ(media::VideoFrame::YV12, frame->format());
     EXPECT_LE(
         0,
         buffer_pool_->RecognizeReservedBuffer(frame->shared_memory_handle()));
     uint8 yuv[3];
     for (int plane = 0; plane < 3; ++plane) {
       yuv[plane] = frame->data(plane)[0];
     }
     // TODO(nick): We just look at the first pixel presently, because if
     // the analysis is too slow, the backlog of frames will grow without bound
     // and trouble erupts. http://crbug.com/174519
     PostColorOrError(SkColorSetRGB(yuv[0], yuv[1], yuv[2]));
   }
 
+  virtual void OnIncomingCapturedEncodedBitstreamBuffer(
+      const scoped_refptr<media::EncodedBitstreamBuffer>& buffer,
+      size_t data_size,
+      base::Time timestamp) OVERRIDE {
+  }

[Ami GONE FROM CHROMIUM, 2013/06/18 18:35:55]

NOTREACHED?
What about actually testing this codepath?

[sheu, 2013/08/22 22:40:31]

NOTREACHED() them.

Testing this codepath is unnecessary until encoded tab capture is implemented.

+
   void PostColorOrError(SkColor new_color) {
     BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, base::Bind(
         &StubConsumer::QuitIfConditionMet, base::Unretained(this), new_color));
   }
 
   virtual void OnError() OVERRIDE {
     {
       base::AutoLock guard(lock_);
       error_encountered_ = true;
     }
     PostColorOrError(kNothingYet);
   }
 
   virtual void OnFrameInfo(const media::VideoCaptureCapability& info) OVERRIDE {
     EXPECT_EQ(kTestWidth, info.width);
     EXPECT_EQ(kTestHeight, info.height);
     EXPECT_EQ(kTestFramesPerSecond, info.frame_rate);
     EXPECT_EQ(media::VideoCaptureCapability::kI420, info.color);
   }
 
+  virtual void OnEncodedFrameInfo(
+      const media::VideoEncodingParameters& params) OVERRIDE {
+  }
+
+  virtual void OnBitstreamConfigChanged(
+      const media::RuntimeVideoEncodingParameters& params) OVERRIDE {
+  }
+
  private:
   base::Lock lock_;
   bool error_encountered_;
   SkColor wait_color_yuv_;
   scoped_refptr<VideoCaptureBufferPool> buffer_pool_;
 
   DISALLOW_COPY_AND_ASSIGN(StubConsumer);
 };
 
 // Test harness that sets up a minimal environment with necessary stubs.
@@ skipping 300 matching lines @@
   ASSERT_NO_FATAL_FAILURE(consumer()->WaitForNextColor(SK_ColorGREEN));
   source()->SetSolidColor(SK_ColorRED);
   ASSERT_NO_FATAL_FAILURE(consumer()->WaitForNextColor(SK_ColorRED));
 
   device()->Stop();
   device()->DeAllocate();
 }
 
 }  // namespace
 }  // namespace content