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

Unified Diff: webrtc/api/video_codecs/video_encoder.cc

Issue 3000693003: Move kMinPixelsPerFrame constant in VideoStreamEncoder to VideoEncoder::ScalingSettings. (Closed)
Patch Set: rebase Created 3 years, 4 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
« no previous file with comments | « webrtc/api/video_codecs/video_encoder.h ('k') | webrtc/media/engine/videoencodersoftwarefallbackwrapper.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/api/video_codecs/video_encoder.cc
diff --git a/webrtc/api/video_codecs/video_encoder.cc b/webrtc/api/video_codecs/video_encoder.cc
index 0da0e60c4518ba9e1e45d7855bdb1727612f153d..f61e4e14f82e25bf3ab9da18cebe0d0d7cccae9f 100644
--- a/webrtc/api/video_codecs/video_encoder.cc
+++ b/webrtc/api/video_codecs/video_encoder.cc
@@ -64,6 +64,17 @@ VideoEncoder::ScalingSettings::ScalingSettings(bool on, int low, int high)
: enabled(on),
thresholds(rtc::Optional<QpThresholds>(QpThresholds(low, high))) {}
+VideoEncoder::ScalingSettings::ScalingSettings(bool on,
+ int low,
+ int high,
+ int min_pixels)
+ : enabled(on),
+ thresholds(rtc::Optional<QpThresholds>(QpThresholds(low, high))),
+ min_pixels_per_frame(min_pixels) {}
+
+VideoEncoder::ScalingSettings::ScalingSettings(bool on, int min_pixels)
+ : enabled(on), min_pixels_per_frame(min_pixels) {}
+
VideoEncoder::ScalingSettings::ScalingSettings(bool on) : enabled(on) {}
VideoEncoder::ScalingSettings::~ScalingSettings() {}
« no previous file with comments | « webrtc/api/video_codecs/video_encoder.h ('k') | webrtc/media/engine/videoencodersoftwarefallbackwrapper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698