Chromium Code Reviews| 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 |