GN: Enable embedder to decide whether or not the external V8 snapshot is enabled.

BUILD.gn

 # Copyright 2014 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.
 
+# Because standalone V8 builds are not supported, assume this is part of a
+# Chromium build.
+import("//build/config/features.gni")
+
 # TODO(jochen): These will need to be user-settable to support standalone V8
 # builds.
 v8_compress_startup_data = "off"
 v8_deprecation_warnings = false
 v8_enable_disassembler = false
 v8_enable_gdbjit = false
 v8_enable_handle_zapping = true
 v8_enable_i18n_support = true
 v8_enable_verify_heap = false
 v8_interpreted_regexp = false
 v8_object_print = false
 v8_postmortem_support = false
 v8_use_snapshot = true
-v8_use_external_startup_data = false
 v8_enable_extra_checks = is_debug
 v8_target_arch = cpu_arch
 v8_random_seed = "314159265"
 
 
 ###############################################################################
 # Configurations
 #
 config("internal_config") {
   visibility = [ ":*" ]  # Only targets in this file can depend on this.
@@ skipping 1376 matching lines @@
 # Public targets
 #
 
 if (component_mode == "shared_library") {
 
 component("v8") {
   sources = [
     "src/v8dll-main.cc",
   ]
 
+  deps = []

[rmcilroy, 2014/11/24 17:21:06]

Is there any reason you need to make these changes?  I would have thought the
if/else if/else structure before would have given you the same result?

[baixo1, 2014/11/24 17:28:11]

I thought the same :). But no :)
When you have the structure:

if (v8_use_external_startup_data)
  do something;
else if (v8_snapshot)
  do something else;
else
  do another thing;

if v8_use_external_startup_data is true, it won't consider the other branches
exist. And then it'll complain that v8_snapshot was defined but never used :)

[rmcilroy, 2014/11/24 18:01:34]

Eww, yuck... Ohh well, perhaps you could just make it 
  if (v8_use_snapshot && v8_use_external_startup_data) {
Since I think v8_use_snapshot has to be true for v8_use_external_startup_data to
be used (and you could assert !v8_use_external_startup_data in the final "else"
branch.

[baixo1, 2014/11/24 21:35:13]

Done.

   if (v8_use_external_startup_data) {
-    deps = [
+    deps += [
       ":v8_base",
       ":v8_external_snapshot",
     ]
-  } else if (v8_use_snapshot) {
-    deps = [
+  }
+  if (v8_use_snapshot && !v8_use_external_startup_data) {
+    deps += [
       ":v8_base",
       ":v8_snapshot",
     ]
-  } else {
-    deps = [
+  }
+  if (!v8_use_snapshot && !v8_use_external_startup_data) {
+    deps += [
       ":v8_base",
       ":v8_nosnapshot",
     ]
   }
 
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [ "//build/config/compiler:no_chromium_code" ]
   configs += [ ":internal_config", ":features", ":toolchain" ]
 
   direct_dependent_configs = [ ":external_config" ]
 
   libs = []
   if (is_android && current_toolchain != host_toolchain) {
     libs += [ "log" ]
   }
 }
 
 } else {
 
 group("v8") {
+  deps = []

[rmcilroy, 2014/11/24 17:21:06]

ditto

   if (v8_use_external_startup_data) {
-    deps = [
+    deps += [
       ":v8_base",
       ":v8_external_snapshot",
     ]
-  } else if (v8_use_snapshot) {
-    deps = [
+  }
+  if (v8_use_snapshot && !v8_use_external_startup_data) {
+    deps += [
       ":v8_base",
       ":v8_snapshot",
     ]
-  } else {
-    deps = [
+  }
+  if (!v8_use_snapshot && !v8_use_external_startup_data) {
+    deps += [
       ":v8_base",
       ":v8_nosnapshot",
     ]
   }
 
   direct_dependent_configs = [ ":external_config" ]
 }
 
 }