Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(367)

Unified Diff: content/renderer/media/media_stream_video_source_unittest.cc

Issue 129923002: Implements MediaStreamVideoSource. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 baca55f7235d7e340f849be39d1533b8cdf83727..5797b15cb564da4a547c9a1847cbfcb564baacf5 100644
--- a/content/renderer/media/media_stream_video_source_unittest.cc
+++ b/content/renderer/media/media_stream_video_source_unittest.cc
@@ -4,7 +4,9 @@
#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"
@@ -14,13 +16,20 @@ class MediaStreamVideoSourceTest
: public ::testing::Test,
public MediaStreamVideoSource {
public:
- MediaStreamVideoSourceTest() {}
+ MediaStreamVideoSourceTest() : MediaStreamVideoSource(&factory_) {
+ factory_.EnsurePeerConnectionFactory();
+ Init();
+ }
+
+ protected:
+ MockMediaStreamDependencyFactory factory_;
};
-TEST_F(MediaStreamVideoSourceTest, OnVideoFrame) {
+TEST_F(MediaStreamVideoSourceTest, DeliverVideoFrame) {
blink::WebMediaConstraints constraints;
- blink::WebMediaStreamTrack track;
- AddTrack(track, constraints);
+ blink::WebMediaStreamTrack webkit_track =
+ factory_.CreateMediaStreamVideoTrack("123");
+ AddTrack(webkit_track, constraints);
SetReadyState(blink::WebMediaStreamSource::ReadyStateLive);
const int kWidth = 640;
const int kHeight = 480;
@@ -29,7 +38,7 @@ TEST_F(MediaStreamVideoSourceTest, OnVideoFrame) {
ASSERT_TRUE(frame.get());
DeliverVideoFrame(frame);
SetReadyState(blink::WebMediaStreamSource::ReadyStateEnded);
- RemoveTrack(track);
+ RemoveTrack(webkit_track);
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698