Adds MediaStreamSource, MediaStreamAudioSource and MediaStreamVideoCaptureDeviceSource

content/renderer/media/media_stream_video_source_unittest.cc

 // Copyright 2014 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 <string>
 
 #include "base/strings/utf_string_conversions.h"
 #include "content/renderer/media/media_stream_video_source.h"
 #include "content/renderer/media/mock_media_stream_dependency_factory.h"
 #include "media/base/video_frame.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace content {
 
 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) {
     MediaStreamVideoSource::DeliverVideoFrame(frame);
   }
 };
 
 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,
                               base::UTF8ToUTF16("dummy_source_name"));
     webkit_source_.setExtraData(new DummyMediaStreamVideoSource(&factory_));
   }
 
  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());
     MockVideoSource* adapter =
         static_cast<MockVideoSource*>(source->GetAdapter());
     EXPECT_EQ(width, adapter->GetLastFrameWidth());
     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;
   scoped_refptr<media::VideoFrame> frame =
       media::VideoFrame::CreateBlackFrame(gfx::Size(kWidth, kHeight));
   ASSERT_TRUE(frame.get());
   source->OnNewFrame(frame);
   VerifyFrame(640, 480, 1);
   source->OnNewFrame(frame);
   VerifyFrame(640, 480, 2);
   source->RemoveTrack(track);
 }
 
 }  // namespace content