Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(444)

Side by Side Diff: content/app/BUILD.gn

Issue 1413893003: Move v8_use_external_startup_data out of //build/config:feature_flags. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove_v8_module_args
Patch Set: Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/chrome_build.gni") 5 import("//build/config/chrome_build.gni")
6 import("//build_overrides/v8.gni")
6 7
7 content_app_sources = [ 8 content_app_sources = [
8 "android/app_jni_registrar.cc", 9 "android/app_jni_registrar.cc",
9 "android/app_jni_registrar.h", 10 "android/app_jni_registrar.h",
10 "android/child_process_service.cc", 11 "android/child_process_service.cc",
11 "android/child_process_service.h", 12 "android/child_process_service.h",
12 "android/content_jni_onload.cc", 13 "android/content_jni_onload.cc",
13 "android/content_main.cc", 14 "android/content_main.cc",
14 "android/content_main.h", 15 "android/content_main.h",
15 "android/library_loader_hooks.cc", 16 "android/library_loader_hooks.cc",
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 "//mojo/environment:chromium", 66 "//mojo/environment:chromium",
66 "//third_party/mojo/src/mojo/edk/system", 67 "//third_party/mojo/src/mojo/edk/system",
67 ] 68 ]
68 } 69 }
69 70
70 content_app_extra_configs = [ 71 content_app_extra_configs = [
71 "//build/config/compiler:wexit_time_destructors", 72 "//build/config/compiler:wexit_time_destructors",
72 "//content:content_implementation", 73 "//content:content_implementation",
73 ] 74 ]
74 75
76 if (v8_use_external_startup_data) {
77 content_app_extra_configs += [ "//v8:external_startup_data" ]
78 }
79
75 if (!is_multi_dll_chrome) { 80 if (!is_multi_dll_chrome) {
76 content_app_deps += [ "//content/gpu:gpu_sources" ] 81 content_app_deps += [ "//content/gpu:gpu_sources" ]
77 } 82 }
78 83
79 # This includes the app sources for both the browser and child processes. 84 # This includes the app sources for both the browser and child processes.
80 source_set("both") { 85 source_set("both") {
81 # Only the public target should depend on this. All other targets (even 86 # Only the public target should depend on this. All other targets (even
82 # internal content ones) should depend on the public one. 87 # internal content ones) should depend on the public one.
83 visibility = [ "//content/public/app:*" ] 88 visibility = [ "//content/public/app:*" ]
84 89
(...skipping 19 matching lines...) Expand all
104 source_set("child") { 109 source_set("child") {
105 visibility = [ "//content/public/app:child" ] 110 visibility = [ "//content/public/app:child" ]
106 111
107 sources = content_app_sources 112 sources = content_app_sources
108 configs += content_app_extra_configs 113 configs += content_app_extra_configs
109 deps = content_app_deps 114 deps = content_app_deps
110 115
111 defines = [ "CHROME_MULTIPLE_DLL_CHILD" ] 116 defines = [ "CHROME_MULTIPLE_DLL_CHILD" ]
112 } 117 }
113 } 118 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698