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

Side by Side Diff: chrome/BUILD.gn

Issue 2070903002: Add an XPC service to handle alert notifications on mac (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@move_builder_add_response
Patch Set: rebased Created 4 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/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 454 matching lines...) Expand 10 before | Expand all | Expand 10 after
465 } 465 }
466 466
467 if (is_chrome_branded && is_official_build) { 467 if (is_chrome_branded && is_official_build) {
468 deps += [ 468 deps += [
469 ":chrome_dsym_archive", 469 ":chrome_dsym_archive",
470 ":chrome_dump_syms", 470 ":chrome_dump_syms",
471 ] 471 ]
472 } 472 }
473 } 473 }
474 474
475 bundle_data("chrome_framework_services") {
476 sources = [
477 "$root_out_dir/AlertNotificationService.xpc",
478 ]
479
480 outputs = [
481 "{{bundle_root_dir}}/XPCServices/{{source_file_part}}",
482 ]
483
484 public_deps = [
485 "//chrome/browser/ui/cocoa/notifications:alert_notification_xpc_service",
486 ]
487 }
488
475 tweak_info_plist("chrome_app_plist") { 489 tweak_info_plist("chrome_app_plist") {
476 info_plist = "app/app-Info.plist" 490 info_plist = "app/app-Info.plist"
477 _keystone_arg = "0" 491 _keystone_arg = "0"
478 if (enable_mac_keystone) { 492 if (enable_mac_keystone) {
479 _keystone_arg = "1" 493 _keystone_arg = "1"
480 } 494 }
481 args = [ 495 args = [
482 "--breakpad=0", 496 "--breakpad=0",
483 "--keystone=$_keystone_arg", 497 "--keystone=$_keystone_arg",
484 "--scm=1", 498 "--scm=1",
(...skipping 500 matching lines...) Expand 10 before | Expand all | Expand 10 after
985 ] 999 ]
986 1000
987 public_deps = [ 1001 public_deps = [
988 ":chrome_dll", 1002 ":chrome_dll",
989 ] 1003 ]
990 1004
991 deps = [ 1005 deps = [
992 ":chrome_framework_helpers", 1006 ":chrome_framework_helpers",
993 ":chrome_framework_plugins", 1007 ":chrome_framework_plugins",
994 ":chrome_framework_resources", 1008 ":chrome_framework_resources",
1009 ":chrome_framework_services",
Robert Sesek 2016/09/29 14:23:04 Adding this deps should be guarded by the buildfla
Miguel Garcia 2016/10/03 16:07:50 What problem do you see in adding this right now?
Robert Sesek 2016/10/04 16:04:25 This will start bundling the XPC service even in t
995 ":packed_resources", 1010 ":packed_resources",
996 ":widevine_cdm_library", 1011 ":widevine_cdm_library",
997 "//build/config/sanitizers:deps", 1012 "//build/config/sanitizers:deps",
998 "//chrome/app/nibs:chrome_xibs", 1013 "//chrome/app/nibs:chrome_xibs",
999 ] 1014 ]
1000 1015
1001 if (is_chrome_branded) { 1016 if (is_chrome_branded) {
1002 deps += [ ":default_apps" ] 1017 deps += [ ":default_apps" ]
1003 } 1018 }
1004 1019
(...skipping 519 matching lines...) Expand 10 before | Expand all | Expand 10 after
1524 "//chrome/app/theme/$branding_path_component/product_logo_48.png", 1539 "//chrome/app/theme/$branding_path_component/product_logo_48.png",
1525 "//chrome/tools/build/linux/chrome-wrapper", 1540 "//chrome/tools/build/linux/chrome-wrapper",
1526 "//third_party/xdg-utils/scripts/xdg-mime", 1541 "//third_party/xdg-utils/scripts/xdg-mime",
1527 "//third_party/xdg-utils/scripts/xdg-settings", 1542 "//third_party/xdg-utils/scripts/xdg-settings",
1528 ] 1543 ]
1529 outputs = [ 1544 outputs = [
1530 "$root_out_dir/{{source_file_part}}", 1545 "$root_out_dir/{{source_file_part}}",
1531 ] 1546 ]
1532 } 1547 }
1533 } 1548 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/ui/cocoa/notifications/BUILD.gn » ('j') | chrome/browser/ui/cocoa/notifications/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698