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

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

Issue 2790823002: Spec compliant video constraints for getUserMedia behind flag. (Closed)
Patch Set: rebase Created 3 years, 8 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/remote_media_stream_impl.cc
diff --git a/content/renderer/media/remote_media_stream_impl.cc b/content/renderer/media/remote_media_stream_impl.cc
index b56c2abd81a9619b9c7851dc6bdbce99d5e5bb0c..431393d0ddc303dccf97b3b8789b7c0269ee763a 100644
--- a/content/renderer/media/remote_media_stream_impl.cc
+++ b/content/renderer/media/remote_media_stream_impl.cc
@@ -160,13 +160,9 @@ class RemoteVideoTrackAdapter
new MediaStreamRemoteVideoSource(std::move(observer)));
InitializeWebkitTrack(blink::WebMediaStreamSource::TypeVideo);
webkit_track()->source().setExtraData(video_source.get());
- // Initial constraints must be provided to a MediaStreamVideoTrack. But
- // no constraints are available initially on a remote video track.
- blink::WebMediaConstraints constraints;
- constraints.initialize();
- MediaStreamVideoTrack* media_stream_track =
- new MediaStreamVideoTrack(video_source.release(), constraints,
- MediaStreamVideoSource::ConstraintsCallback(), enabled);
+ MediaStreamVideoTrack* media_stream_track = new MediaStreamVideoTrack(
+ video_source.release(), MediaStreamVideoSource::ConstraintsCallback(),
+ enabled);
webkit_track()->setTrackData(media_stream_track);
}
};

Powered by Google App Engine
This is Rietveld 408576698