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

Unified Diff: chrome/android/BUILD.gn

Issue 2371843002: Reland of Move language pak files to assets. (Closed)
Patch Set: Fix ContentShell context setPrivateDataPrefix that was breaking tests 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 697030c8b385f04375c048b954a2858da7c75be8..fcf028fed554a6b822c8731be66abbda756b175c 100644
--- a/chrome/android/BUILD.gn
+++ b/chrome/android/BUILD.gn
@@ -67,24 +67,12 @@ 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/policy:app_restrictions_resources",
@@ -457,14 +445,6 @@ 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",
@@ -481,6 +461,7 @@ 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",
@@ -488,6 +469,19 @@ 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",
@@ -564,11 +558,15 @@ if (current_toolchain == default_toolchain) {
}
}
- locale_pak_resources("monochrome_locale_paks") {
- sources = []
+ android_assets("monochrome_pak_assets") {
+ sources = [
+ "$target_gen_dir/monochrome_paks/chrome_100_percent.pak",
+ "$target_gen_dir/monochrome_paks/resources.pak",
+ ]
foreach(_locale, locales) {
sources += [ "$target_gen_dir/monochrome_paks/locales/$_locale.pak" ]
}
+ disable_compression = true
deps = [
":monochrome_paks",
@@ -576,17 +574,6 @@ 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
@@ -642,11 +629,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