Add framerate to VideoSinkWants and ability to signal on overuse

webrtc/test/video_capturer.cc

Index: webrtc/test/video_capturer.cc
diff --git a/webrtc/test/video_capturer.cc b/webrtc/test/video_capturer.cc
new file mode 100644
index 0000000000000000000000000000000000000000..a8347388e87de62b55f18a73dd1befaf0d80db76
--- /dev/null
+++ b/webrtc/test/video_capturer.cc
@@ -0,0 +1,54 @@
+/*
+ *  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.
+ */
+
+#include "webrtc/test/video_capturer.h"
+
+#include "webrtc/base/basictypes.h"
+#include "webrtc/base/constructormagic.h"
+
+namespace webrtc {
+namespace test {
+VideoCapturer::VideoCapturer() : video_adapter_(new cricket::VideoAdapter()) {}
+VideoCapturer::~VideoCapturer() {}
+
+rtc::Optional<VideoFrame> VideoCapturer::AdaptFrame(const VideoFrame& frame) {
+  rtc::Optional<VideoFrame> out_frame;
+  int cropped_width = 0;
+  int cropped_height = 0;
+  int out_width = 0;
+  int out_height = 0;
+
+  if (!video_adapter_->AdaptFrameResolution(
+          frame.width(), frame.height(), frame.timestamp_us() * 1000,
+          &cropped_width, &cropped_height, &out_width, &out_height)) {
+    return out_frame;
+  }

[stefan-webrtc, 2017/03/21 15:37:36]

Looks like we are adapting the resolution here, and then scaling below, is that
because we crop here? Shouldn't the method say so in that case, or at least
maybe we should have a comment?

[sprang_webrtc, 2017/03/21 15:59:18]

No, if we return here out_frame is not set - ie drop frame. I'll add a comment.

+
+  if (out_height != frame.height() || out_width != frame.width()) {
+    rtc::scoped_refptr<I420Buffer> scaled_buffer =
+        I420Buffer::Create(out_width, out_height);
+    scaled_buffer->ScaleFrom(*frame.video_frame_buffer().get());
+    out_frame.emplace(
+        VideoFrame(scaled_buffer, kVideoRotation_0, frame.timestamp_us()));
+  } else {
+    out_frame.emplace(frame);
+  }
+
+  return out_frame;
+}
+
+void VideoCapturer::AddOrUpdateSink(rtc::VideoSinkInterface<VideoFrame>* sink,
+                                    const rtc::VideoSinkWants& wants) {
+  video_adapter_->OnResolutionFramerateRequest(
+      wants.target_pixel_count, wants.max_pixel_count, wants.max_framerate_fps);
+}
+
+}  // namespace test
+}  // namespace webrtc