commit 692409f8e7576ebb46f0f63e45c7a9c34dfe64ce
author Mirko Bonadei <mbonadei@webrtc.org> Thu Jul 19 07:20:53 2018
committer Commit Bot <commit-bot@chromium.org> Thu Jul 19 08:15:52 2018
tree 6bf0b0dd37d97df47829152500277bab162d2a80
parent ddad1fa5f966493d51860cfbebd27cbd23a9dcf1

Enabling clang::find_bad_constructs in modules/BUILD.gn.

This CL removes //build/config/clang:find_bad_constructs from the
suppressed_configs list, which means that clang:find_bad_constructs
is now enabled on these translation units.

Bug: webrtc:9251, webrtc:163
Change-Id: I4d5e8476dca16030814a01447b1d8522f0105b2a
Reviewed-on: https://webrtc-review.googlesource.com/89580
Reviewed-by: Oleh Prypin <oprypin@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24038}

modules/BUILD.gn

diff --git a/modules/BUILD.gn b/modules/BUILD.gn
index 035df66..0109cf2 100644
--- a/modules/BUILD.gn
+++ b/modules/BUILD.gn
@@ -120,11 +120,6 @@
     if (is_ios) {
       deps += [ ":modules_tests_bundle_data" ]
     }
-
-    if (!build_with_chromium && is_clang) {
-      # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
-      suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
-    }
   }
 
   modules_unittests_resources = [
@@ -245,11 +240,6 @@
       "module_common_types_unittest.cc",
     ]
 
-    if (!build_with_chromium && is_clang) {
-      # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
-      suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
-    }
-
     deps = [
       ":module_api",
       "../test:test_main",