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

Side by Side Diff: chrome/BUILD.gn

Issue 2624163003: Remainder of V8 removal (Closed)
Patch Set: . Created 3 years, 11 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
« no previous file with comments | « no previous file | chrome/DEPS » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/buildflag_header.gni") 5 import("//build/buildflag_header.gni")
6 import("//build/config/chrome_build.gni") 6 import("//build/config/chrome_build.gni")
7 import("//build/config/compiler/compiler.gni") 7 import("//build/config/compiler/compiler.gni")
8 import("//build/config/compiler/pgo/pgo.gni") 8 import("//build/config/compiler/pgo/pgo.gni")
9 import("//build/config/features.gni") 9 import("//build/config/features.gni")
10 import("//build/config/locales.gni") 10 import("//build/config/locales.gni")
(...skipping 1203 matching lines...) Expand 10 before | Expand all | Expand 10 after
1214 } 1214 }
1215 } 1215 }
1216 1216
1217 if (!is_component_build) { 1217 if (!is_component_build) {
1218 assert_no_deps = [ 1218 assert_no_deps = [
1219 # Blink and V8 should not be used in the browser process. In component 1219 # Blink and V8 should not be used in the browser process. In component
1220 # build this is OK because all of content is linked into one library. 1220 # build this is OK because all of content is linked into one library.
1221 # Note that the blink_headers target is OK, so we can't do a wildcard for 1221 # Note that the blink_headers target is OK, so we can't do a wildcard for
1222 # all blink targets. 1222 # all blink targets.
1223 "//third_party/WebKit/public:blink", 1223 "//third_party/WebKit/public:blink",
1224 # TODO(brettw) bug 581766: V8 should not be linked into the browser 1224
1225 # process, and then we can enable this. When testing whether you can 1225 # V8 shouldn't be in the browser, however, one header only target
1226 # enable this, run with a non-component build. 1226 # containing the version is allowed, //v8:v8_version. Because of this, we
1227 #"//v8/*", 1227 # can't disallow //v8:* here which we would otherwise want.
1228 "//v8:v8",
1228 ] 1229 ]
1229 } 1230 }
1230 } 1231 }
1231 1232
1232 group("child_dependencies") { 1233 group("child_dependencies") {
1233 public_deps = [ 1234 public_deps = [
1234 "//chrome/browser/devtools", 1235 "//chrome/browser/devtools",
1235 "//chrome/child", 1236 "//chrome/child",
1236 "//chrome/common", 1237 "//chrome/common",
1237 "//chrome/gpu", 1238 "//chrome/gpu",
(...skipping 355 matching lines...) Expand 10 before | Expand all | Expand 10 after
1593 "//chrome/app/theme/$branding_path_component/product_logo_48.png", 1594 "//chrome/app/theme/$branding_path_component/product_logo_48.png",
1594 "//chrome/tools/build/linux/chrome-wrapper", 1595 "//chrome/tools/build/linux/chrome-wrapper",
1595 "//third_party/xdg-utils/scripts/xdg-mime", 1596 "//third_party/xdg-utils/scripts/xdg-mime",
1596 "//third_party/xdg-utils/scripts/xdg-settings", 1597 "//third_party/xdg-utils/scripts/xdg-settings",
1597 ] 1598 ]
1598 outputs = [ 1599 outputs = [
1599 "$root_out_dir/{{source_file_part}}", 1600 "$root_out_dir/{{source_file_part}}",
1600 ] 1601 ]
1601 } 1602 }
1602 } 1603 }
OLDNEW
« no previous file with comments | « no previous file | chrome/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698