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

Unified Diff: webrtc/media/engine/webrtcvideoengine2.cc

Issue 1836043004: Cleanup the VideoAdapter (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 9 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: webrtc/media/engine/webrtcvideoengine2.cc
diff --git a/webrtc/media/engine/webrtcvideoengine2.cc b/webrtc/media/engine/webrtcvideoengine2.cc
index 9f74a86a8480a1ec7b47948090a875ed2ce6a537..8cd21473c3462dfc7f01e8da1e2b4be74b390ebf 100644
--- a/webrtc/media/engine/webrtcvideoengine2.cc
+++ b/webrtc/media/engine/webrtcvideoengine2.cc
@@ -1825,7 +1825,7 @@ void WebRtcVideoChannel2::WebRtcVideoSendStream::SetSendParameters(
<< "RecreateWebRtcStream (send) because of SetSendParameters";
RecreateWebRtcStream();
}
- } // release |lock_|
+ } // release |lock_|
// |capturer_->AddOrUpdateSink| may not be called while holding |lock_| since
// that might cause a lock order inversion.
@@ -2016,8 +2016,8 @@ void WebRtcVideoChannel2::WebRtcVideoSendStream::OnLoadUpdate(Load load) {
rtc::Optional<int> max_pixel_count;
rtc::Optional<int> max_pixel_count_step_up;
if (load == kOveruse) {
- max_pixel_count = rtc::Optional<int>(
- (last_dimensions_.height * last_dimensions_.width) / 2);
+ max_pixel_count = rtc::Optional<int>(
+ (last_dimensions_.height * last_dimensions_.width * 3) / 5);
// Increase |number_of_cpu_adapt_changes_| if
// sink_wants_.max_pixel_count will be changed since
// last time |capturer_->AddOrUpdateSink| was called. That is, this will
@@ -2078,9 +2078,8 @@ WebRtcVideoChannel2::WebRtcVideoSendStream::GetVideoSenderInfo() {
stats = stream_->GetStats();
}
info.adapt_changes = number_of_cpu_adapt_changes_;
- info.adapt_reason = cpu_restricted_counter_ <= 0
- ? CoordinatedVideoAdapter::ADAPTREASON_NONE
- : CoordinatedVideoAdapter::ADAPTREASON_CPU;
+ info.adapt_reason =
+ cpu_restricted_counter_ <= 0 ? ADAPTREASON_NONE : ADAPTREASON_CPU;
if (capturer_) {
VideoFormat last_captured_frame_format;
@@ -2094,7 +2093,7 @@ WebRtcVideoChannel2::WebRtcVideoSendStream::GetVideoSenderInfo() {
// higher video layer(s) can be dropped due to bitrate constraints.
// Note, adapt_changes only include changes from the video_adapter.
if (stats.bw_limited_resolution)
- info.adapt_reason |= CoordinatedVideoAdapter::ADAPTREASON_BANDWIDTH;
+ info.adapt_reason |= ADAPTREASON_BANDWIDTH;
info.encoder_implementation_name = stats.encoder_implementation_name;
info.ssrc_groups = ssrc_groups_;

Powered by Google App Engine
This is Rietveld 408576698