Refactor VideoDestinationHandler to implement MediaStreamVideoSource.

content/renderer/media/webrtc/video_destination_handler_unittest.cc

Index: content/renderer/media/webrtc/video_destination_handler_unittest.cc
diff --git a/content/renderer/media/video_destination_handler_unittest.cc b/content/renderer/media/webrtc/video_destination_handler_unittest.cc
similarity index 39%
rename from content/renderer/media/video_destination_handler_unittest.cc
rename to content/renderer/media/webrtc/video_destination_handler_unittest.cc
index f220f9f5e7fc0b0d32de890f6e9d2338e2f368d2..7a99b38e82929bea10fcf46c5f992d4f82598f7e 100644
--- a/content/renderer/media/video_destination_handler_unittest.cc
+++ b/content/renderer/media/webrtc/video_destination_handler_unittest.cc
@@ -6,58 +6,24 @@
 
 #include "base/strings/utf_string_conversions.h"
 #include "content/renderer/media/media_stream.h"
+#include "content/renderer/media/media_stream_video_track.h"
 #include "content/renderer/media/mock_media_stream_dependency_factory.h"
 #include "content/renderer/media/mock_media_stream_registry.h"
-#include "content/renderer/media/video_destination_handler.h"
-#include "testing/gmock/include/gmock/gmock.h"
+#include "content/renderer/media/mock_media_stream_video_sink.h"
+#include "content/renderer/media/webrtc/video_destination_handler.h"
+#include "content/renderer/pepper/pepper_plugin_instance_impl.h"
+#include "content/renderer/pepper/ppb_image_data_impl.h"
+#include "content/test/ppapi_unittest.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/WebKit/public/platform/WebMediaStreamTrack.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 
-using cricket::CapturedFrame;
-using cricket::CaptureState;
-using cricket::VideoCapturer;
-using cricket::VideoFormat;
-using cricket::VideoFormatPod;
-
 namespace content {
 
 static const std::string kTestStreamUrl = "stream_url";
 static const std::string kUnknownStreamUrl = "unknown_stream_url";
-static const VideoFormatPod kTestFormat = {
-  640, 360, FPS_TO_INTERVAL(30), cricket::FOURCC_ANY
-};
-
-class PpFrameWriterTest
-    : public ::testing::Test,
-      public sigslot::has_slots<> {
- public:
-  PpFrameWriterTest()
-      : last_capture_state_(cricket::CS_FAILED),
-        captured_frame_count_(0),
-        captured_frame_(NULL) {
-    writer_.SignalStateChange.connect(this, &PpFrameWriterTest::OnStateChange);
-    writer_.SignalFrameCaptured.connect(
-        this, &PpFrameWriterTest::OnFrameCaptured);
-  }
-
-  void OnStateChange(VideoCapturer* capturer, CaptureState state) {
-    last_capture_state_ = state;
-  }
 
-  void OnFrameCaptured(VideoCapturer* capturer, const CapturedFrame* frame) {
-    ++captured_frame_count_;
-    captured_frame_ = const_cast<CapturedFrame*>(frame);
-  }
-
- protected:
-  PpFrameWriter writer_;
-  CaptureState last_capture_state_;
-  int captured_frame_count_;
-  CapturedFrame* captured_frame_;
-};
-
-class VideoDestinationHandlerTest : public ::testing::Test {
+class VideoDestinationHandlerTest : public PpapiUnittest {
  public:
   VideoDestinationHandlerTest() : registry_(&factory_) {
     registry_.Init(kTestStreamUrl);
@@ -68,33 +34,6 @@ class VideoDestinationHandlerTest : public ::testing::Test {
   MockMediaStreamRegistry registry_;
 };
 
-TEST_F(PpFrameWriterTest, StartStop) {
-  EXPECT_FALSE(writer_.IsRunning());
-  EXPECT_EQ(cricket::CS_STARTING, writer_.Start(VideoFormat(kTestFormat)));
-  EXPECT_TRUE(writer_.IsRunning());
-  EXPECT_EQ(cricket::CS_FAILED, writer_.Start(VideoFormat(kTestFormat)));
-  writer_.Stop();
-  EXPECT_EQ(cricket::CS_STOPPED, last_capture_state_);
-}
-
-TEST_F(PpFrameWriterTest, GetPreferredFourccs) {
-  std::vector<uint32> fourccs;
-  EXPECT_TRUE(writer_.GetPreferredFourccs(&fourccs));
-  EXPECT_EQ(1u, fourccs.size());
-  EXPECT_EQ(cricket::FOURCC_BGRA, fourccs[0]);
-}
-
-TEST_F(PpFrameWriterTest, GetBestCaptureFormat) {
-  VideoFormat desired(kTestFormat);
-  VideoFormat best_format;
-  EXPECT_FALSE(writer_.GetBestCaptureFormat(desired, NULL));
-  EXPECT_TRUE(writer_.GetBestCaptureFormat(desired, &best_format));
-  EXPECT_EQ(cricket::FOURCC_BGRA, best_format.fourcc);
-
-  desired.fourcc = best_format.fourcc;
-  EXPECT_EQ(desired, best_format);
-}
-
 TEST_F(VideoDestinationHandlerTest, Open) {
   FrameWriterInterface* frame_writer = NULL;
   // Unknow url will return false.
@@ -102,24 +41,42 @@ TEST_F(VideoDestinationHandlerTest, Open) {
                                              kUnknownStreamUrl, &frame_writer));
   EXPECT_TRUE(VideoDestinationHandler::Open(&factory_, &registry_,
                                             kTestStreamUrl, &frame_writer));
-  EXPECT_TRUE(frame_writer);
+  // The |frame_writer| is a proxy and is owned by who call Open.
+  delete frame_writer;
+}
+
+TEST_F(VideoDestinationHandlerTest, PutFrame) {
+  FrameWriterInterface* frame_writer = NULL;
+  EXPECT_TRUE(VideoDestinationHandler::Open(&factory_, &registry_,
+                                            kTestStreamUrl, &frame_writer));
+  ASSERT_TRUE(frame_writer);
 
   // Verify the video track has been added.
   const blink::WebMediaStream test_stream = registry_.test_stream();
   blink::WebVector<blink::WebMediaStreamTrack> video_tracks;
   test_stream.videoTracks(video_tracks);
-  EXPECT_EQ(1u, video_tracks.size());
+  ASSERT_EQ(1u, video_tracks.size());
 
   // Verify the native video track has been added.
-  MediaStream* native_stream = MediaStream::GetMediaStream(test_stream);
-  DCHECK(native_stream);
-  webrtc::MediaStreamInterface* webrtc_stream =
-      MediaStream::GetAdapter(test_stream);
-  DCHECK(webrtc_stream);
-  webrtc::VideoTrackVector webrtc_video_tracks =
-      webrtc_stream->GetVideoTracks();
-  EXPECT_EQ(1u, webrtc_video_tracks.size());
+  MediaStreamVideoTrack* native_track =
+      MediaStreamVideoTrack::GetVideoTrack(video_tracks[0]);
+  ASSERT_TRUE(native_track != NULL);
+
+  MockMediaStreamVideoSink sink;
+  native_track->AddSink(&sink);
+
+  scoped_refptr<PPB_ImageData_Impl> image(
+      new PPB_ImageData_Impl(instance()->pp_instance(),
+                             PPB_ImageData_Impl::ForTest()));
+  image->Init(PP_IMAGEDATAFORMAT_BGRA_PREMUL, 640, 360, true);
+  frame_writer->PutFrame(image, 10);
+  EXPECT_EQ(1, sink.number_of_frames());
+  // TODO(perkj): Verify that the track output I420 when
+  // https://codereview.chromium.org/213423006/ is landed.
+
+  native_track->RemoveSink(&sink);
 
+  // The |frame_writer| is a proxy and is owned by who call Open.
   delete frame_writer;
 }