Let ViEEncoder express resolution requests as Sinkwants

webrtc/call/bitrate_estimator_tests.cc

Index: webrtc/call/bitrate_estimator_tests.cc
diff --git a/webrtc/call/bitrate_estimator_tests.cc b/webrtc/call/bitrate_estimator_tests.cc
index 80949baba4d5933d672511385076e4c8b7ae5560..203beddf8227dc1ce7fba131fc84e6b5619056ff 100644
--- a/webrtc/call/bitrate_estimator_tests.cc
+++ b/webrtc/call/bitrate_estimator_tests.cc
@@ -178,7 +178,7 @@ class BitrateEstimatorTest : public test::CallTest {
       frame_generator_capturer_.reset(test::FrameGeneratorCapturer::Create(
           kDefaultWidth, kDefaultHeight, kDefaultFramerate,
           Clock::GetRealTimeClock()));
-      send_stream_->SetSource(frame_generator_capturer_.get());
+      send_stream_->SetSource(frame_generator_capturer_.get(), false);

[kthelgason, 2016/10/18 08:31:20]

I personally really dislike boolean arguments like this. There is no way to know
what the "false" means but looking up the definition of the function. IMO this
would be a lot clearer with a boolean enum, something like:

enum CpuAdaptation : bool {
    Off = true,
    On = false,
  };

Then this call would become:
SetSource(capturer, CpuAdaptation::On);

So much clearer. Whether it's worth the hassle on the other hand...

[perkj_webrtc, 2016/10/26 16:40:16]

ok- I agree, I added an enum that match the current spec and hope we can later
implement all aspects of it.  For now it is translated in webrtcvideoengine2.

       send_stream_->Start();
       frame_generator_capturer_->Start();