Move kMinPixelsPerFrame constant in VideoStreamEncoder to VideoEncoder::ScalingSettings.

webrtc/api/video_codecs/video_encoder.cc

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))),

[kthelgason, 2017/08/16 13:42:22]

i prefer to defer to the other constructor when possible, but up to you. I know
it's not widely used in webrtc

+      min_pixels_per_frame(min_pixels) {}
+
+VideoEncoder::ScalingSettings::ScalingSettings(bool on, int min_pixels)
+    : enabled(on), min_pixels_per_frame(min_pixels) {}

[kthelgason, 2017/08/16 13:42:22]

same here.

+
 VideoEncoder::ScalingSettings::ScalingSettings(bool on) : enabled(on) {}
 
 VideoEncoder::ScalingSettings::~ScalingSettings() {}