Mojo C++ bindings GYP build: only unset "-Wglobal-constructors' for unittests...

Mojo C++ bindings GYP build: only unset "-Wglobal-constructors' for unittests depending on WTF types.

BUG=583738

Review-Url: https://codereview.chromium.org/2038413003
Cr-Commit-Position: refs/heads/master@{#398162}

mojo/mojo_edk_tests.gyp

@@ -41,6 +41,9 @@
         'mojo_public.gyp:mojo_public_test_interfaces_struct_traits',
         'mojo_public.gyp:mojo_public_test_utils',
       ],
+      'variables': {
+        'clang_warning_flags_unset': [ '-Wglobal-constructors' ],
+      },
       'sources': [
         'public/cpp/bindings/tests/array_common_test.h',
         'public/cpp/bindings/tests/array_unittest.cc',
@@ -92,6 +95,9 @@
         'public/cpp/bindings/tests/variant_test_util.h',
       ],
       'conditions': [
+        # TODO(yzshen): Blink-flavor bindings tests should be moved into
+        # mojo_public_bindings_for_blink_tests (which should eventually be moved
+        # into blink).
         ['OS=="ios"', {
           'dependencies!': [
             'mojo_public.gyp:mojo_public_test_interfaces_blink',
@@ -120,6 +126,9 @@
         'mojo_public.gyp:mojo_public_test_wtf_types',
         'mojo_public.gyp:mojo_public_test_wtf_types_blink',
       ],
+      'variables': {
+         'clang_warning_flags_unset': [ '-Wglobal-constructors' ],
+      },
       'sources': [
         'public/cpp/bindings/tests/array_common_test.h',
         'public/cpp/bindings/tests/container_test_util.cc',

mojo/mojo_public.gyp

@@ -238,6 +238,7 @@
       ],
     },
     {
+      # TODO(yzshen): crbug.com/617718 Consider moving this into blink.
       # GN version: //mojo/public/cpp/bindings:wtf_support
       'target_name': 'mojo_cpp_bindings_wtf_support',
       'type': 'static_library',
@@ -261,11 +262,6 @@
         'mojo_cpp_bindings',
         '../third_party/WebKit/Source/config.gyp:config',
       ],
-      'direct_dependent_settings': {
-        'variables': {
-          'clang_warning_flags_unset': [ '-Wglobal-constructors' ],
-        },
-      },
     },
     {
       # GN version: //mojo/message_pump

mojo/public/cpp/bindings/BUILD.gn

@@ -146,6 +146,7 @@ source_set("struct_traits") {
 }
 
 if (!is_ios) {
+  # TODO(yzshen): crbug.com/617718 Consider moving this into blink.
   source_set("wtf_support") {
     sources = [
       "array_traits_wtf.h",