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

build/config/BUILD.gn

Index: build/config/BUILD.gn
diff --git a/build/config/BUILD.gn b/build/config/BUILD.gn
index 4af73895467c840e757efb0330248daab44c5961..97d7d04c8f245b433d46e61c923c3922b6252d7d 100644
--- a/build/config/BUILD.gn
+++ b/build/config/BUILD.gn
@@ -8,7 +8,7 @@ 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() {