ui: accessibility: remove -Wunused-function for Atk

ui/accessibility/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+import("//build/config/linux/pkg_config.gni")
 import("//build/config/ui.gni")
 import("//build/json_schema_api.gni")
 import("//testing/test.gni")
 if (is_android) {
   import("//build/config/android/rules.gni")
 }
 
+pkg_config("atk") {

[Dirk Pranke, 2015/07/08 19:33:05]

this and the above change probably need to be protected by if (use_x11) { ... },
no? Or possible if (is_linux), not sure what the right dependency is.

[brettw, 2015/07/08 20:11:17]

Yeah, looks like x11 is the correct condition.

[llandwerlin-old, 2015/07/08 20:21:07]

It was under the x11 condition in build/config/linux/BUILD.gn.
Will change

+  packages = [ "atk" ]
+  atk_lib_dir = exec_script(pkg_config_script,
+                            [
+                              "--libdir",
+                              "atk",
+                            ],
+                            "string")
+  defines = [ "ATK_LIB_DIR=\"$atk_lib_dir\"" ]
+}
+
+# gn orders flags on a target before flags from configs. The default config
+# adds -Wall, and these flags have to be after -Wall -- so they need to come
+# from a config and can't be on the target directly.
+config("atk_warnings") {
+  cflags = [
+    # glib uses the pre-c++11 typedef-as-static_assert hack.
+    "-Wno-unused-local-typedef",
+
+    # G_DEFINE_TYPE automatically generates a *get_instance_private
+    # inline function after glib 2.37. That's unused. Prevent to
+    # complain about it.
+    "-Wno-unused-function",
+  ]
+}
+
 component("accessibility") {
   sources = [
     "ax_node.cc",
     "ax_node.h",
     "ax_node_data.cc",
     "ax_node_data.h",
     "ax_serializable_tree.cc",
     "ax_serializable_tree.h",
     "ax_text_utils.cc",
     "ax_text_utils.h",
@@ skipping 32 matching lines @@
 
   if (use_aura && !is_chromeos && is_linux && use_x11) {
     sources += [
       "platform/atk_util_auralinux.cc",
       "platform/atk_util_auralinux.h",
       "platform/ax_platform_node_auralinux.cc",
       "platform/ax_platform_node_auralinux.h",
     ]
 
     configs += [
-      "//build/config/linux:atk",
-      "//build/config/linux:atk_warnings",
+      ":atk",
+      ":atk_warnings",
       "//build/config/linux:gconf",
       "//build/config/linux:glib",
     ]
   }
 }
 
 if (is_android) {
   android_library("ui_accessibility_java") {
     deps = []
     srcjar_deps = [ ":ax_enumerations_srcjar" ]
@@ skipping 49 matching lines @@
 json_schema_api("ax_gen") {
   sources = [
     "ax_enums.idl",
   ]
   deps = [
     "//base/third_party/dynamic_annotations",
   ]
   root_namespace = "ui"
   schemas = true
 }