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

Unified Diff: docs/android_build_instructions.md

Issue 2611843007: Update incremental install docs. (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: docs/android_build_instructions.md
diff --git a/docs/android_build_instructions.md b/docs/android_build_instructions.md
index ee3c399eeff7cc6cb53f9c3f0c3741e23b0a9ef4..9ebe46563e5c78801b1aec0f875c12c3fdfa9237 100644
--- a/docs/android_build_instructions.md
+++ b/docs/android_build_instructions.md
@@ -1,6 +1,6 @@
# Checking out and building Chromium for Android
-There are instructions for other platforms linked from the
+There are instructions for other platforms linked from the
[get the code](get_the_code.md) page.
## Instructions for Google Employees
@@ -311,11 +311,12 @@ For information on running tests, see [android\_test\_instructions.md](https://c
### Faster Edit/Deploy (GN only)
GN's "incremental install" uses reflection and side-loading to speed up the edit
-& deploy cycle (normally < 10 seconds).
+& deploy cycle (normally < 10 seconds). The initial launch of the apk will be
+a little slower since updated dex files are installed manually.
* Make sure to set` is_component_build = true `in your GN args
* All apk targets have \*`_incremental` targets defined (e.g.
- `chrome_public_apk_incremental`)
+ `chrome_public_apk_incremental`) except for Webview and Monochrome
Here's an example:
@@ -345,6 +346,27 @@ To uninstall:
out/Default/bin/install_chrome_public_apk_incremental -v --uninstall
```
+A subtly erroneous flow arises when you build a regular apk but install an
+incremental apk (e.g.
+```ninja -C out/Default foo_apk out/Default/bin/install_foo_apk_incremental```).
agrieve 2017/01/05 16:26:40 nit: these commands need either a && or a ; to sep
estevenson 2017/01/05 16:36:48 Done.
+Setting incremental_apk_by_default = true in your GN args aliases regular
agrieve 2017/01/05 16:26:39 nit: add `` arround incremental_apk_by_default = t
estevenson 2017/01/05 16:36:47 Done.
+targets as their incremental counterparts. With this arg set, the commands
+above become:
+
+```shell
+ninja -C out/Default chrome_public_apk
+out/Default/bin/install_chrome_public_apk
+
+ninja -C out/Default base_unittests
+out/Default/bin/run_base_unittests
+
+ninja -C out/Default chrome_public_test_apk
+out/Default/bin/run_chrome_public_test_apk
+```
+
+If you want to build a non-incremental apk you'll need to remove
+```incremental_apk_by_default``` from your GN args.
+
## Tips, tricks, and troubleshooting
### Rebuilding libchrome.so for a particular release
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698