Bitrate controller for VideoToolbox encoder.

webrtc/modules/remote_bitrate_estimator/BUILD.gn

Index: webrtc/modules/remote_bitrate_estimator/BUILD.gn
diff --git a/webrtc/modules/remote_bitrate_estimator/BUILD.gn b/webrtc/modules/remote_bitrate_estimator/BUILD.gn
index 99c297dda61f920f9571d5226e43ea7cfc92cb98..4734be585563017666f69809e4645e76ee69c2d3 100644
--- a/webrtc/modules/remote_bitrate_estimator/BUILD.gn
+++ b/webrtc/modules/remote_bitrate_estimator/BUILD.gn
@@ -10,8 +10,6 @@ source_set("remote_bitrate_estimator") {
   sources = [
     "include/bwe_defines.h",
     "include/remote_bitrate_estimator.h",
-    "rate_statistics.cc",
-    "rate_statistics.h",
   ]
 
   configs += [ "../../:common_inherited_config" ]
@@ -19,6 +17,7 @@ source_set("remote_bitrate_estimator") {
   deps = [
     ":rbe_components",
     "../..:webrtc_common",
+    "../../base:rtc_base",

[pbos-webrtc, 2016/02/19 11:26:39]

Doesn't this cause Chromium to depend on rtc_base? Shouldn't this be
rtc_base_approved?

[tkchin_webrtc, 2016/02/19 18:51:09]

If that is the case, what do you propose to do with ratestatistics.h? Presumably
I can't add that to rtc_base_approved. The alternative is to have this module
depend on remote_bitrate_estimator..

[pbos-webrtc, 2016/02/19 18:57:49]

I see nothing scary in rate_statistics that wouldn't permit it to be in
rtc_base_approved, move it there. It's already in there in the gyp.

[tkchin_webrtc, 2016/02/19 23:30:04]

Done.

     "../../system_wrappers",
   ]
 }