Bitrate controller for VideoToolbox encoder.

webrtc/modules/remote_bitrate_estimator/BUILD.gn

 # 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.
 
 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" ]
 
   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",
   ]
 }
 
 source_set("rbe_components") {
   sources = [
     "aimd_rate_control.cc",
     "aimd_rate_control.h",
     "include/send_time_history.h",
     "inter_arrival.cc",
@@ skipping 16 matching lines @@
   deps = [
     "../..:webrtc_common",
   ]
 
   if (is_clang) {
     # Suppress warnings from Chrome's Clang plugins.
     # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
     configs -= [ "//build/config/clang:find_bad_constructs" ]
   }
 }