Let ViEEncoder express resolution requests as Sinkwants

webrtc/test/frame_generator.cc

 /*
  *  Copyright (c) 2013 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.
  */
 #include "webrtc/test/frame_generator.h"
@@ skipping 238 matching lines @@
   rtc::CritScope lock(&crit_);
   if (sink_)
     sink_->OnFrame(video_frame);
 }
 
 void FrameForwarder::AddOrUpdateSink(rtc::VideoSinkInterface<VideoFrame>* sink,
                                      const rtc::VideoSinkWants& wants) {
   rtc::CritScope lock(&crit_);
   RTC_DCHECK(!sink_ || sink_ == sink);
   sink_ = sink;
+  current_sink_wants_ = wants;
 }
 
 void FrameForwarder::RemoveSink(rtc::VideoSinkInterface<VideoFrame>* sink) {
   rtc::CritScope lock(&crit_);
   RTC_DCHECK_EQ(sink, sink_);
   sink_ = nullptr;
 }
 
+rtc::VideoSinkWants FrameForwarder::current_sink_wants() const {
+  rtc::CritScope lock(&crit_);
+  return current_sink_wants_;
+}
+
+size_t FrameForwarder::number_of_sinks() const {

[åsapersson, 2016/10/24 07:54:05]

maybe bool has_sink()

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

A source should support multiple sinks.. Currently though, no unit test need
that and thus this implementation only support one. 
But I guess current_sink_wants does not necessarily make sense if there are
multiple sinks so ok, lets rename this for now and we will see in the future. 

+  rtc::CritScope lock(&crit_);
+  return sink_ ? 1u : 0u;
+}
+
 FrameGenerator* FrameGenerator::CreateChromaGenerator(size_t width,
                                                       size_t height) {
   return new ChromaGenerator(width, height);
 }
 
 FrameGenerator* FrameGenerator::CreateFromYuvFile(
     std::vector<std::string> filenames,
     size_t width,
     size_t height,
     int frame_repeat_count) {
@@ skipping 25 matching lines @@
     files.push_back(file);
   }
 
   return new ScrollingImageFrameGenerator(
       clock, files, source_width, source_height, target_width, target_height,
       scroll_time_ms, pause_time_ms);
 }
 
 }  // namespace test
 }  // namespace webrtc