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

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

Issue 2964003002: Remove support for old getUserMedia video constraints algorithm. (Closed)
Patch Set: rebase and address hbos@ comments 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/user_media_client_impl.cc
diff --git a/content/renderer/media/user_media_client_impl.cc b/content/renderer/media/user_media_client_impl.cc
index 96ebd7bdfa3e904ab5ee720b51fc1a33f03277a6..8cdc27b0e215489368e36c3a5326edc113d29377 100644
--- a/content/renderer/media/user_media_client_impl.cc
+++ b/content/renderer/media/user_media_client_impl.cc
@@ -1010,11 +1010,6 @@ MediaStreamVideoSource* UserMediaClientImpl::CreateVideoSource(
const MediaStreamSource::SourceStoppedCallback& stop_callback) {
DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
DCHECK(current_request_info_);
- if (IsOldVideoConstraints()) {
- return new MediaStreamVideoCapturerSource(stop_callback, device,
- render_frame());
- }
-
DCHECK(current_request_info_->video_capture_settings().HasValue());
return new MediaStreamVideoCapturerSource(
stop_callback, device,
@@ -1484,13 +1479,6 @@ UserMediaClientImpl::UserMediaRequestInfo::CreateAndStartVideoTrack(
DCHECK(native_source);
sources_.push_back(source);
sources_waiting_for_callback_.push_back(native_source);
- if (IsOldVideoConstraints()) {
- return MediaStreamVideoTrack::CreateVideoTrack(
- native_source, request_.VideoConstraints(),
- base::Bind(&UserMediaClientImpl::UserMediaRequestInfo::OnTrackStarted,
- AsWeakPtr()),
- true);
- }
return MediaStreamVideoTrack::CreateVideoTrack(
native_source, video_capture_settings_.track_adapter_settings(),
video_capture_settings_.noise_reduction(), is_video_content_capture_,

Powered by Google App Engine
This is Rietveld 408576698