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

Unified Diff: chrome/android/BUILD.gn

Issue 2406093002: Fix monochrome not booting issue (Closed)
Patch Set: 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/config/android/rules.gni ('k') | chrome/android/chrome_public_apk_tmpl.gni » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/BUILD.gn
diff --git a/chrome/android/BUILD.gn b/chrome/android/BUILD.gn
index 2a8b645d27f124df805990b46fa2c6caf38a8708..ee9af9ad23c50efb992e9daa51e968d690339c6e 100644
--- a/chrome/android/BUILD.gn
+++ b/chrome/android/BUILD.gn
@@ -67,12 +67,24 @@ jinja_template("chrome_sync_shell_android_manifest") {
]
}
+locale_pak_resources("chrome_locale_paks") {
+ sources = []
+ foreach(_locale, locales - android_chrome_omitted_locales) {
+ sources += [ "$root_out_dir/locales/$_locale.pak" ]
+ }
+
+ deps = [
+ "//chrome:packed_resources",
+ ]
+}
+
android_resources("chrome_java_resources") {
resource_dirs = [
"java/res",
"//chrome/android/java/res_chromium",
]
deps = [
+ ":chrome_locale_paks",
":chrome_strings_grd",
"//chrome/app:java_strings_grd",
"//components/autofill/android:autofill_java_resources",
@@ -456,6 +468,14 @@ android_resources("chrome_public_apk_resources") {
]
}
+# TODO(agrieve): Delete once downstream no longer references this target.
+java_group("chrome_public_apk_assets") {
+ deps = [
+ ":chrome_public_non_pak_assets",
+ ":chrome_public_pak_assets",
+ ]
+}
+
java_group("chrome_public_non_pak_assets") {
deps = [
"//chrome/android/webapk/libs/runtime_library:runtime_library_assets",
@@ -472,7 +492,6 @@ android_assets("chrome_public_pak_assets") {
disable_compression = true
deps = [
- ":chrome_public_locale_pak_assets",
"//chrome:packed_resources",
"//chrome/android/webapk/libs/runtime_library:runtime_library_assets",
"//third_party/icu:icu_assets",
@@ -480,19 +499,6 @@ android_assets("chrome_public_pak_assets") {
]
}
-# This target is separate from chrome_public_pak_assets because it does not
-# disable compression.
-android_assets("chrome_public_locale_pak_assets") {
- sources = []
- foreach(_locale, locales - android_chrome_omitted_locales) {
- sources += [ "$root_out_dir/locales/$_locale.pak" ]
- }
-
- deps = [
- "//chrome:packed_resources",
- ]
-}
-
jinja_template_resources("chrome_public_apk_template_resources") {
resources = [
"java/res_template/xml/chromebackupscheme.xml",
@@ -569,15 +575,11 @@ if (current_toolchain == default_toolchain) {
}
}
- android_assets("monochrome_pak_assets") {
- sources = [
- "$target_gen_dir/monochrome_paks/chrome_100_percent.pak",
- "$target_gen_dir/monochrome_paks/resources.pak",
- ]
+ locale_pak_resources("monochrome_locale_paks") {
+ sources = []
foreach(_locale, locales) {
sources += [ "$target_gen_dir/monochrome_paks/locales/$_locale.pak" ]
}
- disable_compression = true
deps = [
":monochrome_paks",
@@ -585,6 +587,17 @@ if (current_toolchain == default_toolchain) {
}
} # current_toolchain == host_toolchain
+android_assets("monochrome_pak_assets") {
+ sources = [
+ "$target_gen_dir/monochrome_paks/chrome_100_percent.pak",
+ "$target_gen_dir/monochrome_paks/resources.pak",
+ ]
+ deps = [
+ ":monochrome_paks",
+ ]
+ disable_compression = true
+}
+
#
# Only 32-bit //chrome/android/monochrome is needed, beside
# being built with 32-bit default toolchain, it is also built
@@ -640,11 +653,11 @@ template("chrome_public_apk_tmpl_shared") {
forward_variables_from(invoker, "*")
native_lib_version_rule = "//build/util:chrome_version_json"
+
deps += [
":chrome_java",
+ ":chrome_public_apk_assets",
":chrome_public_apk_resources",
- ":chrome_public_non_pak_assets",
- ":chrome_public_pak_assets",
"//base:base_java",
]
}
« no previous file with comments | « build/config/android/rules.gni ('k') | chrome/android/chrome_public_apk_tmpl.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698