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

Unified Diff: ios/chrome/app/BUILD.gn

Issue 2632333002: Add a prototype today extension behind gn flag (Closed)
Patch Set: renaming 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 side-by-side diff with in-line comments
Download patch
Index: ios/chrome/app/BUILD.gn
diff --git a/ios/chrome/app/BUILD.gn b/ios/chrome/app/BUILD.gn
index c94edaee7214bbb08849d60e0b9973611c6bfab1..100e547ef5ea0084c88d1a2bb556f1052c12ed05 100644
--- a/ios/chrome/app/BUILD.gn
+++ b/ios/chrome/app/BUILD.gn
@@ -273,12 +273,16 @@ ios_app_bundle("chrome") {
bundle_deps = [ "//ios/chrome/app/resources" ]
- if (ios_enable_today_extension && current_toolchain == default_toolchain) {
- deps += [ ":today_extension_bundle" ]
- }
-
- if (ios_enable_share_extension && current_toolchain == default_toolchain) {
- deps += [ ":share_extension_bundle" ]
+ if (current_toolchain == default_toolchain) {
+ if (ios_enable_today_extension) {
+ deps += [ ":today_extension_bundle" ]
+ }
+ if (ios_enable_widget_extension) {
+ deps += [ ":widget_extension_bundle" ]
+ }
+ if (ios_enable_share_extension) {
+ deps += [ ":share_extension_bundle" ]
+ }
}
extra_substitutions = [
@@ -298,31 +302,49 @@ ios_app_bundle("chrome") {
}
}
-if (ios_enable_today_extension && current_toolchain == default_toolchain) {
- bundle_data("today_extension_bundle") {
- public_deps = [
- "//ios/chrome/today_extension",
- ]
- sources = [
- "$root_out_dir/today_extension.appex",
- ]
- outputs = [
- "{{bundle_plugins_dir}}/{{source_file_part}}",
- ]
+if (current_toolchain == default_toolchain) {
+ if (ios_enable_today_extension) {
+ bundle_data("today_extension_bundle") {
+ public_deps = [
+ "//ios/chrome/today_extension",
+ ]
+ sources = [
+ "$root_out_dir/today_extension.appex",
+ ]
+ outputs = [
+ "{{bundle_plugins_dir}}/{{source_file_part}}",
+ ]
+ }
+ }
+
+ if (ios_enable_widget_extension) {
+ bundle_data("widget_extension_bundle") {
+ public_deps = [
+ "//ios/chrome/widget_extension",
+ ]
+ sources = [
+ "$root_out_dir/widget_extension.appex",
+ ]
+
+ # The output is renamed today_extension.appex so that signing in canary works and clobbering is not necessary when switching between this extension and the today extension.
sdefresne 2017/01/18 11:11:35 Please reformat your comment to fit in 80 characte
lody 2017/01/18 14:56:39 Done.
+ outputs = [
+ "{{bundle_plugins_dir}}/today_extension.appex",
Olivier 2017/01/18 11:54:08 I am a little worried to have bundle name (today_e
lody 2017/01/18 14:56:39 Yes I did :P
+ ]
+ }
}
-}
-if (ios_enable_share_extension && current_toolchain == default_toolchain) {
- bundle_data("share_extension_bundle") {
- public_deps = [
- "//ios/chrome/share_extension",
- ]
- sources = [
- "$root_out_dir/share_extension.appex",
- ]
- outputs = [
- "{{bundle_plugins_dir}}/{{source_file_part}}",
- ]
+ if (ios_enable_share_extension) {
+ bundle_data("share_extension_bundle") {
+ public_deps = [
+ "//ios/chrome/share_extension",
+ ]
+ sources = [
+ "$root_out_dir/share_extension.appex",
+ ]
+ outputs = [
+ "{{bundle_plugins_dir}}/{{source_file_part}}",
+ ]
+ }
}
}

Powered by Google App Engine
This is Rietveld 408576698