Conditionally convert V8 build overrides to declare_args.

gni/v8.gni

 # Copyright 2016 the V8 project 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/sanitizers/sanitizers.gni")
 import("//build/config/v8_target_cpu.gni")
 import("//build_overrides/v8.gni")
 
 declare_args() {
   # Includes files needed for correctness fuzzing.
@@ skipping 17 matching lines @@
   v8_use_snapshot = true
 
   # Use external files for startup data blobs:
   # the JS builtins sources and the start snapshot.
   v8_use_external_startup_data = ""
 
   # Enable ECMAScript Internationalization API. Enabling this feature will
   # add a dependency on the ICU library.
   v8_enable_i18n_support = true
 
-  # Enable inspector. See include/v8-inspector.h.
-  v8_enable_inspector = v8_enable_inspector_override
+  # TODO(brettw) http://crbug.com/684096 Remove the define condition and the
+  # v8_enable_inspector_override variable when the build_override conversion is
+  # complete. This value should just default to true.
+  if (defined(v8_enable_inspector_override)) {
+    # Enable inspector. See include/v8-inspector.h.
+    v8_enable_inspector = v8_enable_inspector_override
+  } else {
+    # Enable inspector. See include/v8-inspector.h.
+    v8_enable_inspector = true
+  }
+
+  # Use static libraries instead of source_sets.
+  v8_static_library = false
 }
 
 if (v8_use_external_startup_data == "") {
   # If not specified as a gn arg, use external startup data by default if
   # a snapshot is used and if we're not on ios.
   v8_use_external_startup_data = v8_use_snapshot && !is_ios
 }
 
 if (v8_enable_backtrace == "") {
   v8_enable_backtrace = is_debug && !v8_optimized_debug
@@ skipping 79 matching lines @@
 }
 
 template("v8_component") {
   component(target_name) {
     forward_variables_from(invoker, "*", [ "configs" ])
     configs += invoker.configs
     configs -= v8_remove_configs
     configs += v8_add_configs
   }
 }