Remove -Wno-unused-private-field clang warning suppression.

third_party/hunspell/BUILD.gn

Index: third_party/hunspell/BUILD.gn
diff --git a/third_party/hunspell/BUILD.gn b/third_party/hunspell/BUILD.gn
index 3219c9db3ed0f7ea4be5192f39cef64128797d0d..9e9f249e5b31b59ee0ca12a8768d4e9db942da67 100644
--- a/third_party/hunspell/BUILD.gn
+++ b/third_party/hunspell/BUILD.gn
@@ -51,8 +51,17 @@ source_set("hunspell") {
     "src/parsers/textparser.hxx",
   ]
 
+  config("hunspell_warnings") {

[brettw, 2015/07/09 18:03:07]

This should go at the top (don't nest targets).

[Nico, 2015/07/09 18:18:46]

Almost all of the -Wno targets are nested.
(https://code.google.com/p/chromium/codesearch#search/&q=pcre:yes%20config.*%5...
– that doesn't find all, but some)

[benwells, 2015/07/10 00:17:44]

So this got landed as is in my sleep-time. I am happy to unnest if that is the
right thing to do in a follow up. Let me know if I should.

+    if (is_clang) {
+      cflags = [ "-Wno-unused-private-field" ]
+    }
+  }
+
   configs -= [ "//build/config/compiler:chromium_code" ]
-  configs += [ "//build/config/compiler:no_chromium_code" ]
+  configs += [
+    "//build/config/compiler:no_chromium_code",
+    ":hunspell_warnings",

[brettw, 2015/07/09 18:03:07]

Ditto

+  ]
   public_configs = [ ":hunspell_config" ]
 
   defines = [ "OPENOFFICEORG" ]