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

Unified Diff: content/renderer/media/webrtc/media_stream_remote_video_source.cc

Issue 2972553002: Remove MediaStreamVideoSource::GetCurrentSupportedFormats() (Closed)
Patch Set: remove extra use in media_stream_video_capturer_source_unittest.cc Created 3 years, 5 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/webrtc/media_stream_remote_video_source.cc
diff --git a/content/renderer/media/webrtc/media_stream_remote_video_source.cc b/content/renderer/media/webrtc/media_stream_remote_video_source.cc
index bf34d0479871bc13c3d9ca7ab6af496ce8e5e2b4..5e28c7408746df715aa067b11246103146f498f3 100644
--- a/content/renderer/media/webrtc/media_stream_remote_video_source.cc
+++ b/content/renderer/media/webrtc/media_stream_remote_video_source.cc
@@ -186,18 +186,6 @@ void MediaStreamRemoteVideoSource::OnSourceTerminated() {
StopSourceImpl();
}
-void MediaStreamRemoteVideoSource::GetCurrentSupportedFormats(
- int max_requested_width,
- int max_requested_height,
- double max_requested_frame_rate,
- const VideoCaptureDeviceFormatsCB& callback) {
- DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
- media::VideoCaptureFormats formats;
- // Since the remote end is free to change the resolution at any point in time
- // the supported formats are unknown.
- callback.Run(formats);
-}
-
void MediaStreamRemoteVideoSource::StartSourceImpl(
const media::VideoCaptureFormat& format,
const blink::WebMediaConstraints& constraints,

Powered by Google App Engine
This is Rietveld 408576698