Move //build/module_args/v8.gni to //build_overrides.

build/config/BUILD.gn

 # Copyright (c) 2013 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/allocator.gni")
 import("//build/config/chrome_build.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/sanitizers/sanitizers.gni")
 import("//build/config/ui.gni")
-import("//build/module_args/v8.gni")
+import("//build_overrides/v8.gni")

[kjellander_chromium, 2015/10/13 07:31:35]

Having this being included in the //build tree will enforce us to create a dummy
v8.gni to put into our own //build_overrides directory of WebRTC, which feels
wrong.
It's not a big deal, but it's likely to break for us every time a new file is
introduced (other projects etc).

Without it, I get a failure about that file being missing when I tried to roll
past this CL. See https://codereview.webrtc.org/1403453003/#ps1 for context
(I'll add a build_overrides dir as a workaround in PS#2).

[Dirk Pranke, 2015/10/13 17:11:41]

It should feel wrong, because it probably is wrong :). I think, ideally, //build
should not import or depend on anything outside of //build.

However, we're not there yet, and we're going to have to land a bunch of
CLs to clean things up.

In the meantime, yeah, you'll have to create a dummy v8.gni. Hopefully we can
avoid needing any others and get rid of this one.

 import("//build/toolchain/goma.gni")
 
 declare_args() {
   # When set (the default) enables C++ iterator debugging in debug builds.
   # Iterator debugging is always off in release builds (technically, this flag
   # affects the "debug" config, which is always available but applied by
   # default only in debug builds).
   #
   # Iterator debugging is generally useful for catching bugs. But it can
   # introduce extra locking to check the state of an iterator against the state
@@ skipping 427 matching lines @@
     # This error doesn't happen every time. In VS2013, it seems if the .pch
     # file doesn't exist, no error will be generated (probably MS tested this
     # case but forgot the other one?). To reproduce this error, do a build,
     # then delete the precompile.c.obj file, then build again.
     cflags_c = [ "/wd4206" ]
   } else if (is_mac && !is_official_build && !use_goma) {
     precompiled_header = "build/precompile.h"
     precompiled_source = "//build/precompile.h"
   }
 }