Adds MediaStreamSource, MediaStreamAudioSource and MediaStreamVideoCaptureDeviceSource

content/renderer/media/media_stream_video_source_unittest.cc

Index: content/renderer/media/media_stream_video_source_unittest.cc
diff --git a/content/renderer/media/media_stream_video_source_unittest.cc b/content/renderer/media/media_stream_video_source_unittest.cc
index 946707bcdd159be6255f8cee047a5b9697b870ba..6ae837a4fefff1edcb6d405ef600a0a5aca2e146 100644
--- a/content/renderer/media/media_stream_video_source_unittest.cc
+++ b/content/renderer/media/media_stream_video_source_unittest.cc
@@ -16,13 +16,10 @@ class DummyMediaStreamVideoSource : public MediaStreamVideoSource {
  public:
   DummyMediaStreamVideoSource(MediaStreamDependencyFactory* factory)
       : MediaStreamVideoSource(factory) {
-    Init();
-    SetVideoSource(GetAdapter());
-    SetReadyState(blink::WebMediaStreamSource::ReadyStateLive);
+    blink::WebMediaConstraints constraints;

[Ronghua Wu (Left Chromium), 2014/01/16 23:02:37]

remove

[perkj_chrome, 2014/01/17 13:19:45]

Done.

   }
 
   virtual ~DummyMediaStreamVideoSource() {
-    SetReadyState(blink::WebMediaStreamSource::ReadyStateEnded);
   }
 
   void OnNewFrame(const scoped_refptr<media::VideoFrame>& frame) {
@@ -33,7 +30,9 @@ class DummyMediaStreamVideoSource : public MediaStreamVideoSource {
 class MediaStreamVideoSourceTest
     : public ::testing::Test {
  public:
-  MediaStreamVideoSourceTest() {
+  MediaStreamVideoSourceTest()
+      : number_of_successfull_constraints_applied_(0),
+        number_of_failed_constraints_applied_(0) {
     factory_.EnsurePeerConnectionFactory();
     webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"),
                               blink::WebMediaStreamSource::TypeVideo,
@@ -43,12 +42,31 @@ class MediaStreamVideoSourceTest
 
  protected:
   // Create a track that's associated with |webkit_source_|.
-  blink::WebMediaStreamTrack CreateTrack(const std::string& id) {
+  blink::WebMediaStreamTrack CreateTrack(
+      const std::string& id,
+      const blink::WebMediaConstraints& constraints) {
     blink::WebMediaStreamTrack track;
     track.initialize(base::UTF8ToUTF16(id), webkit_source_);
+
+    DummyMediaStreamVideoSource* source =
+        static_cast<DummyMediaStreamVideoSource*>(track.source().extraData());
+
+    source->AddTrack(track,
+                     constraints,
+                     base::Bind(
+                         &MediaStreamVideoSourceTest::OnConstraintsApplied,
+                         base::Unretained(this)));
     return track;
   }
 
+  void StartAdapter() {
+    factory_.last_video_source()->SetLive();
+  }
+
+  void FailToStartAdapter() {
+    factory_.last_video_source()->SetEnded();
+  }
+
   void VerifyFrame(int width, int height, int num) {
     DummyMediaStreamVideoSource* source =
       static_cast<DummyMediaStreamVideoSource*>(webkit_source_.extraData());
@@ -58,17 +76,69 @@ class MediaStreamVideoSourceTest
     EXPECT_EQ(height, adapter->GetLastFrameHeight());
     EXPECT_EQ(num, adapter->GetFrameNum());
   }
+
+  int NumberOfSuccessConstraintsCallbacks() {
+    return number_of_successfull_constraints_applied_;
+  }
+
+  int NumberOfFailedConstraintsCallbacks() {
+    return number_of_failed_constraints_applied_;
+  }
+
  private:
+  void OnConstraintsApplied(MediaStreamSource* source, bool success) {
+    ASSERT_EQ(source, webkit_source_.extraData());
+
+    if (success)
+      ++number_of_successfull_constraints_applied_;
+    else
+      ++number_of_failed_constraints_applied_;
+  }
+
+  int number_of_successfull_constraints_applied_;
+  int number_of_failed_constraints_applied_;
   MockMediaStreamDependencyFactory factory_;
   blink::WebMediaStreamSource webkit_source_;
 };
 
+TEST_F(MediaStreamVideoSourceTest, AddTrackAndStartAdapter) {
+  blink::WebMediaConstraints constraints;
+  blink::WebMediaStreamTrack track = CreateTrack("123", constraints);
+  StartAdapter();
+  EXPECT_EQ(1, NumberOfSuccessConstraintsCallbacks());
+}
+
+TEST_F(MediaStreamVideoSourceTest, AddTwoTracksBeforeAdapterStart) {
+  blink::WebMediaConstraints constraints;
+  blink::WebMediaStreamTrack track1 = CreateTrack("123", constraints);
+  blink::WebMediaStreamTrack track2 = CreateTrack("123", constraints);
+  EXPECT_EQ(0, NumberOfSuccessConstraintsCallbacks());
+  StartAdapter();
+  EXPECT_EQ(2, NumberOfSuccessConstraintsCallbacks());
+}
+
+TEST_F(MediaStreamVideoSourceTest, AddTrackAfterAdapterStart) {
+  blink::WebMediaConstraints constraints;
+  blink::WebMediaStreamTrack track1 = CreateTrack("123", constraints);
+  StartAdapter();
+  EXPECT_EQ(1, NumberOfSuccessConstraintsCallbacks());
+  blink::WebMediaStreamTrack track2 = CreateTrack("123", constraints);
+  EXPECT_EQ(2, NumberOfSuccessConstraintsCallbacks());
+}
+
+TEST_F(MediaStreamVideoSourceTest, AddTrackAndFailToStartAdapter) {
+  blink::WebMediaConstraints constraints;
+  blink::WebMediaStreamTrack track = CreateTrack("123", constraints);
+  FailToStartAdapter();
+  EXPECT_EQ(1, NumberOfFailedConstraintsCallbacks());
+}
+
 TEST_F(MediaStreamVideoSourceTest, DeliverVideoFrame) {
   blink::WebMediaConstraints constraints;
-  blink::WebMediaStreamTrack track = CreateTrack("123");
+  blink::WebMediaStreamTrack track = CreateTrack("123", constraints);
+  StartAdapter();

[Ronghua Wu (Left Chromium), 2014/01/16 23:02:37]

we don't need to start adapter in the real case, right? If that's the case, we
should setlive the mock source by default

[perkj_chrome, 2014/01/17 13:19:45]

I guess that depend on the webrtc::MediaStreamVideoSource implementation. The
current video capture implementation is set to live when the camera has been
started. I think it do that when the first video frame is received.
I added it here to test what happens if the source fail to start. Once the
"real" don't use cricket::VideoCapturer we must have another way of testing what
happen if the video capture device dont start. 
I changed the method name to hopefully better reflect the purpose.

   DummyMediaStreamVideoSource* source =
       static_cast<DummyMediaStreamVideoSource*>(track.source().extraData());
-  source->AddTrack(track, constraints);
   VerifyFrame(0, 0, 0);
   const int kWidth = 640;
   const int kHeight = 480;