Chrome MediaStream VideoTrack implementation.

content/renderer/media/media_stream_dependency_factory_unittest.cc

Index: content/renderer/media/media_stream_dependency_factory_unittest.cc
diff --git a/content/renderer/media/media_stream_dependency_factory_unittest.cc b/content/renderer/media/media_stream_dependency_factory_unittest.cc
index 7ed5c218fb63e4709621147761ab7b526ed92ea4..4a10db8b219c320b386b44c68a841660759e8da6 100644
--- a/content/renderer/media/media_stream_dependency_factory_unittest.cc
+++ b/content/renderer/media/media_stream_dependency_factory_unittest.cc
@@ -105,8 +105,21 @@ class MediaStreamDependencyFactoryTest : public ::testing::Test {
     for (size_t i = 0; i < video_track_vector.size(); ++i) {
       video_track_vector[i].initialize(video_sources[i].id(),
                                        video_sources[i]);
-      video_track_vector[i].setExtraData(
-          new MediaStreamVideoTrack(dependency_factory_.get()));
+      MediaStreamVideoSource* native_source =
+          MediaStreamVideoSource::GetVideoSource(video_sources[i]);
+      MediaStreamVideoTrack* native_video_track =
+          new MediaStreamVideoTrack(
+              native_source,

[Jói, 2014/02/28 14:52:50]

nit: this and next parameter would fit on previous line

[perkj_chrome, 2014/03/02 09:20:44]

Done.

+              dependency_factory_.get());
+      video_track_vector[i].setExtraData(native_video_track);
+      blink::WebMediaConstraints constraints;
+      constraints.initialize();
+      native_source->AddTrack(video_track_vector[i],
+                              constraints,
+                              MediaStreamVideoSource::ConstraintsCallback());
+

[Alpha Left Google, 2014/02/28 02:49:22]

nit: remove these empty lines.

[perkj_chrome, 2014/03/02 09:20:44]

Done.

+
+
     }
 
     stream_desc.initialize("media stream", audio_track_vector,