Move usage of QualityScaler to ViEEncoder.

webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter.cc

 /*
  *  Copyright (c) 2014 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 453 matching lines @@
 bool SimulcastEncoderAdapter::SupportsNativeHandle() const {
   // We should not be calling this method before streaminfos_ are configured.
   RTC_DCHECK(!streaminfos_.empty());
   for (const auto& streaminfo : streaminfos_) {
     if (!streaminfo.encoder->SupportsNativeHandle())
       return false;
   }
   return true;
 }
 
+QualityScaler::Settings SimulcastEncoderAdapter::GetQPThresholds() const {
+  // Turn off quality scaling for simulcast
+  RTC_DCHECK(!streaminfos_.empty()) << "Encoder must be initialized first.";

[magjed_webrtc, 2016/10/26 14:28:36]

Is it a security risk if we execute streaminfos_[0].encoder->GetQPThresholds()
when it's empty? Then this should be a CHECK. Or you can just handle the empty
case like:
return NumberOfStreams(codec_) == 1 ? streaminfos_[0].encoder->GetQPThresholds()
: QualityScaler::Settings(false);

[kthelgason, 2016/10/26 19:02:49]

Done.

+  if (NumberOfStreams(codec_) > 1)
+    return QualityScaler::Settings(false);
+  else
+    return streaminfos_[0].encoder->GetQPThresholds();
+}
+
 const char* SimulcastEncoderAdapter::ImplementationName() const {
   return implementation_name_.c_str();
 }
 
 }  // namespace webrtc