Move kMinPixelsPerFrame constant in VideoStreamEncoder to VideoEncoder::ScalingSettings.

webrtc/api/video_codecs/video_encoder.cc

 /*
  *  Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 46 matching lines @@
   h264_settings.ppsLen = 0;
   h264_settings.profile = H264::kProfileConstrainedBaseline;
 
   return h264_settings;
 }
 
 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() {}
 
 
 int32_t VideoEncoder::SetRates(uint32_t bitrate, uint32_t framerate) {
   RTC_NOTREACHED() << "SetRate(uint32_t, uint32_t) is deprecated.";
   return -1;
 }
 
@@ skipping 12 matching lines @@
 }
 
 bool VideoEncoder::SupportsNativeHandle() const {
   return false;
 }
 
 const char* VideoEncoder::ImplementationName() const {
   return "unknown";
 }
 }  // namespace webrtc