Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/android/BUILD.gn

Issue 29684695: Issue 6287 - Create codename Orca v.0 to launch internally to the team (Closed)
Patch Set: Fixed comments Created Feb. 2, 2018, 12:50 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/config/android/version.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 d3e4b8a592ad711c2274eaaa7b2f7e371f624c80..07bcc2a75dda1ec7e7b9af2ab7c7ab4823a905ba 100644
--- a/chrome/android/BUILD.gn
+++ b/chrome/android/BUILD.gn
@@ -19,7 +19,8 @@ import("//tools/resources/generate_resource_whitelist.gni")
import("channel.gni")
import("java_sources.gni")
-manifest_package = "org.chromium.chrome"
+# Change package name. See https://issues.adblockplus.org/ticket/6287
+manifest_package = "org.adblockplus.orca"
chrome_public_jinja_variables = default_chrome_public_jinja_variables +
[ "manifest_package=$manifest_package" ]
@@ -88,7 +89,8 @@ jinja_template("chrome_sync_shell_android_manifest") {
android_resources("chrome_java_resources") {
resource_dirs = [
"java/res",
- "//chrome/android/java/res_chromium",
+ # Change res directory for rebranding. See https://issues.adblockplus.org/ticket/6287
+ "//chrome/android/java/res_orca",
]
deps = [
":chrome_strings_grd",
@@ -648,7 +650,8 @@ if (enable_vr) {
# Overrides icon / name defined in chrome_java_resources.
android_resources("chrome_public_apk_resources") {
- resource_dirs = [ "java/res_chromium" ]
+ # Change res directory for rebranding. See https://issues.adblockplus.org/ticket/6287
+ resource_dirs = [ "java/res_orca" ]
# Dep needed to ensure override works properly.
deps = [
@@ -972,7 +975,8 @@ template("chrome_public_apk_tmpl_shared") {
chrome_public_apk_tmpl_shared("chrome_public_apk") {
android_manifest = chrome_public_android_manifest
android_manifest_dep = ":chrome_public_android_manifest"
- apk_name = "ChromePublic"
+ # Change APK name for rebranding. See https://issues.adblockplus.org/ticket/6287
+ apk_name = "OrcaPublic"
shared_libraries = [ ":chrome" ]
shared_libraries += [ "//third_party/libadblockplus_android:aar_java_native" ]
}
@@ -980,7 +984,8 @@ chrome_public_apk_tmpl_shared("chrome_public_apk") {
chrome_public_apk_tmpl_shared("chrome_modern_public_apk") {
android_manifest = chrome_modern_public_android_manifest
android_manifest_dep = ":chrome_modern_public_android_manifest"
- apk_name = "ChromeModernPublic"
+ # Change APK name for rebranding. See https://issues.adblockplus.org/ticket/6287
+ apk_name = "OrcaModernPublic"
shared_libraries = [ ":chrome" ]
shared_libraries += [ "//third_party/libadblockplus_android:aar_java_native" ]
@@ -1106,7 +1111,8 @@ instrumentation_test_apk("chrome_public_test_apk") {
if (enable_vr) {
instrumentation_test_apk("chrome_public_test_vr_apk") {
- apk_name = "ChromePublicTestVr"
+ # Change APK name for rebranding. See https://issues.adblockplus.org/ticket/6287
+ apk_name = "OrcaPublicTestVr"
apk_under_test = ":chrome_public_apk"
android_manifest = chrome_public_test_vr_apk_manifest
android_manifest_dep = ":chrome_public_test_vr_apk_manifest"
« no previous file with comments | « build/config/android/version.gni ('k') | chrome/android/chrome_public_apk_tmpl.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld