[Mac/iOS/GN] Remove bad -Wno-deprecated-declarations from //third_party/google_toolbox_for_mac publ…

third_party/google_toolbox_for_mac/BUILD.gn

Index: third_party/google_toolbox_for_mac/BUILD.gn
diff --git a/third_party/google_toolbox_for_mac/BUILD.gn b/third_party/google_toolbox_for_mac/BUILD.gn
index 42f04a69cabcddd666e29868f654993925e95b16..fccd6b6c188ee298e44c632ac311aa72f5e2a7a5 100644
--- a/third_party/google_toolbox_for_mac/BUILD.gn
+++ b/third_party/google_toolbox_for_mac/BUILD.gn
@@ -2,7 +2,7 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
-config("google_toolbox_for_mac_config") {
+config("google_toolbox_for_mac_public_config") {
   include_dirs = [
     ".",
     "src",
@@ -10,11 +10,6 @@ config("google_toolbox_for_mac_config") {
     "src/DebugUtils",
     "src/Foundation",
   ]
-
-  # TODO(crbug.com/569158): Suppresses warnings that are treated as errors
-  # when minimum iOS version support is increased to iOS 9 and up.
-  # This should be removed once all deprecation violations have been fixed.
-  cflags = [ "-Wno-deprecated-declarations" ]
 }
 
 component("google_toolbox_for_mac") {
@@ -40,11 +35,8 @@ component("google_toolbox_for_mac") {
   ]
 
   configs -= [ "//build/config/compiler:chromium_code" ]
-  configs += [
-    ":google_toolbox_for_mac_config",
-    "//build/config/compiler:no_chromium_code",
-  ]
-  public_configs = [ ":google_toolbox_for_mac_config" ]
+  configs += [ "//build/config/compiler:no_chromium_code" ]
+  public_configs = [ ":google_toolbox_for_mac_public_config" ]
 
   if (!is_ios) {
     sources += [
@@ -192,6 +184,11 @@ component("google_toolbox_for_mac") {
       "src/iPhone/GTMUILocalizer.h",
       "src/iPhone/GTMUILocalizer.m",
     ]
+
+    # TODO(crbug.com/569158): Suppresses warnings that are treated as errors
+    # when minimum iOS version support is increased to iOS 9 and up.
+    # This should be removed once all deprecation violations have been fixed.
+    cflags = [ "-Wno-deprecated-declarations" ]

[Nico, 2016/06/24 19:17:40]

oh, fyi, in general putting -Wno-foo flags in cflags instead of a config doesn't
work. Since this target removes chromium_code and adds no_chromium_code,
no_chromium_code is pretty late in the config list, and its flags get added
before the cflags here. no_chromium_code does contain -Wall, so -Wall will
appear after -Wno-foo, and if foo is in -Wall then the -Wall will reenable the
warning that -Wno-foo disabled. In this case, Wdeprecated-declarations
apparently isn't in -Wall, which might be why this works. (Or the flag isn't
needed anymore?)

[Robert Sesek, 2016/06/24 19:59:25]

Hm. It is necessary on iOS and not on Mac (see ps1 failures). Should I switch
these to a config for iOS? It currently is working as intended.

   }
 
   if (is_component_build) {