From fc173d00ecf14cd86d29f525a3b9298aa8130b01 Mon Sep 17 00:00:00 2001 From: Will Harris Date: Wed, 29 Aug 2018 13:56:00 -0700 Subject: [PATCH] Add no_size_t_to_int_warning suppression to webrtc. Current set of warnings that need fixing before this warning can be enabled is here -> https://pastebin.com/raw/jTddgPzP BUG=chromium:588506, webrtc:9451 Change-Id: Id7896cf48c7231b2ee28dde378ff3ce17da73c2b Reviewed-on: https://webrtc-review.googlesource.com/96623 Commit-Queue: Will Harris Reviewed-by: Mirko Bonadei Reviewed-by: Karl Wiberg Cr-Commit-Position: refs/heads/master@{#24503} --- modules/congestion_controller/goog_cc/BUILD.gn | 3 --- webrtc.gni | 5 ++++- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/modules/congestion_controller/goog_cc/BUILD.gn b/modules/congestion_controller/goog_cc/BUILD.gn index 79a1d61df2..a5e4bbc798 100644 --- a/modules/congestion_controller/goog_cc/BUILD.gn +++ b/modules/congestion_controller/goog_cc/BUILD.gn @@ -83,9 +83,6 @@ rtc_source_set("estimators") { "trendline_estimator.h", ] - # TODO(jschuh): Bug 1348: fix this warning. - configs += [ "//build/config/compiler:no_size_t_to_int_warning" ] - deps = [ "../../../logging:rtc_event_bwe", "../../../logging:rtc_event_log_api", diff --git a/webrtc.gni b/webrtc.gni index 133f86ce05..c43ebd3e84 100644 --- a/webrtc.gni +++ b/webrtc.gni @@ -238,7 +238,10 @@ webrtc_root = get_path_info(".", "abspath") # It sets defines, include paths and compilation warnings accordingly, # both for WebRTC stand-alone builds and for the scenario when WebRTC # native code is built as part of Chromium. -rtc_common_configs = [ webrtc_root + ":common_config" ] +rtc_common_configs = [ + webrtc_root + ":common_config", + "//build/config/compiler:no_size_t_to_int_warning", +] if (is_mac || is_ios) { rtc_common_configs += [ "//build/config/compiler:enable_arc" ]