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

Side by Side Diff: chrome/BUILD.gn

Issue 2038593002: [Mac/GN] When using debug_devtools=true, bundle the inspector resources using a symlink. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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/test/BUILD.gn » ('j') | chrome/test/BUILD.gn » ('J')
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/features.gni") 8 import("//build/config/features.gni")
9 import("//build/config/locales.gni") 9 import("//build/config/locales.gni")
10 import("//build/config/sanitizers/sanitizers.gni") 10 import("//build/config/sanitizers/sanitizers.gni")
(...skipping 400 matching lines...) Expand 10 before | Expand all | Expand 10 after
411 } 411 }
412 } 412 }
413 } 413 }
414 } else if (is_mac) { 414 } else if (is_mac) {
415 chrome_helper_name = chrome_product_full_name + " Helper" 415 chrome_helper_name = chrome_product_full_name + " Helper"
416 chrome_framework_name = chrome_product_full_name + " Framework" 416 chrome_framework_name = chrome_product_full_name + " Framework"
417 417
418 group("chrome") { 418 group("chrome") {
419 deps = [ 419 deps = [
420 ":chrome_app", 420 ":chrome_app",
421 ":devtools_debug_resources",
421 ] 422 ]
422 } 423 }
423 424
424 tweak_info_plist("chrome_app_plist") { 425 tweak_info_plist("chrome_app_plist") {
425 info_plist = "app/app-Info.plist" 426 info_plist = "app/app-Info.plist"
426 _keystone_arg = "0" 427 _keystone_arg = "0"
427 if (enable_mac_keystone) { 428 if (enable_mac_keystone) {
428 _keystone_arg = "1" 429 _keystone_arg = "1"
429 } 430 }
430 args = [ 431 args = [
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
679 680
680 if (v8_use_external_startup_data) { 681 if (v8_use_external_startup_data) {
681 sources += [ 682 sources += [
682 "$root_out_dir/natives_blob.bin", 683 "$root_out_dir/natives_blob.bin",
683 "$root_out_dir/snapshot_blob.bin", 684 "$root_out_dir/snapshot_blob.bin",
684 ] 685 ]
685 public_deps += [ "//v8" ] 686 public_deps += [ "//v8" ]
686 } 687 }
687 } 688 }
688 689
690 # When debug_devtools is enabled, symlink the inspector resources into the
691 # framework bundle. The resources go into the final output directory for the
692 # framework in the app bundle, rather than the framework bundle in
693 # root_out_dir, since copy_bundle_data copies the contents of the link
694 # rather than the link itself.
695 action("devtools_debug_resources") {
696 _stamp = "$target_out_dir/run_${target_name}.stamp"
697
698 outputs = [
699 _stamp,
700 ]
701
702 # Convert the destination to an absolute path to make symlinking easier
703 # (no pwd manipulation).
704 _symlink_dest = rebase_path(
705 "$root_out_dir/$chrome_product_full_name.app/Contents/Versions/$chro me_version_full/$chrome_framework_name.framework/Resources/inspector")
706
707 _stamp_arg = rebase_path(_stamp, root_out_dir)
708
709 if (debug_devtools) {
710 script = "//build/symlink.py"
711 args = [
712 "-f",
713 "--touch",
714 _stamp_arg,
715
716 # Convert the symlink source to an absolute path, too.
717 rebase_path("$root_out_dir/resources/inspector"),
718 _symlink_dest,
719 ]
720 } else {
721 script = "//build/rmdir_and_stamp.py"
Nico 2016/06/02 18:03:46 rmdir_and_stamp removes the symlink, not the conte
Robert Sesek 2016/06/02 18:20:08 Correct.
Robert Sesek 2016/06/02 18:41:56 Actually, the rmdir is unnecessary because when co
Nico 2016/06/02 18:44:34 I figured the rmdir is here so that if you remove
Robert Sesek 2016/06/02 18:47:11 If you remove devtools_debug, it will create a new
722 args = [
723 _symlink_dest,
724 _stamp_arg,
725 ]
726 }
727
728 deps = [
729 # Depend on :chrome_app to ensure that the bundle is produced before
730 # creating or destroying the symlink.
731 ":chrome_app",
732 "//third_party/WebKit/public:blink_devtools_frontend_resources",
733 ]
734 }
735
689 if (enable_nacl) { 736 if (enable_nacl) {
690 bundle_data("chrome_framework_plugins") { 737 bundle_data("chrome_framework_plugins") {
691 sources = [] 738 sources = []
692 outputs = [ 739 outputs = [
693 "{{bundle_root_dir}}/Internet Plug-Ins/{{source_file_part}}", 740 "{{bundle_root_dir}}/Internet Plug-Ins/{{source_file_part}}",
694 ] 741 ]
695 public_deps = [] 742 public_deps = []
696 743
697 if (enable_nacl) { 744 if (enable_nacl) {
698 sources += [ "$root_out_dir/nacl_irt_x86_64.nexe" ] 745 sources += [ "$root_out_dir/nacl_irt_x86_64.nexe" ]
(...skipping 868 matching lines...) Expand 10 before | Expand all | Expand 10 after
1567 "//chrome/app/theme/$branding_path_component/product_logo_48.png", 1614 "//chrome/app/theme/$branding_path_component/product_logo_48.png",
1568 "//chrome/tools/build/linux/chrome-wrapper", 1615 "//chrome/tools/build/linux/chrome-wrapper",
1569 "//third_party/xdg-utils/scripts/xdg-mime", 1616 "//third_party/xdg-utils/scripts/xdg-mime",
1570 "//third_party/xdg-utils/scripts/xdg-settings", 1617 "//third_party/xdg-utils/scripts/xdg-settings",
1571 ] 1618 ]
1572 outputs = [ 1619 outputs = [
1573 "$root_out_dir/{{source_file_part}}", 1620 "$root_out_dir/{{source_file_part}}",
1574 ] 1621 ]
1575 } 1622 }
1576 } 1623 }
OLDNEW
« no previous file with comments | « no previous file | chrome/test/BUILD.gn » ('j') | chrome/test/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698