[Chromecast] Apply exclude_unwind_tables for appropriate builds

[Chromecast] Apply exclude_unwind_tables for appropriate builds

This reduces our installed APK size from ~90MB to 65 on ATV.  It
reduces earth OTA size by 1.5MB and earth cast_shell size by 5MB.

Leave unwind tables for builds where we regularly use a debugger (x86
and ATV debug).

BUG=internal b/36082628

Review-Url: https://codereview.chromium.org/2770833003
Cr-Commit-Position: refs/heads/master@{#459619}
Committed: https://chromium.googlesource.com/chromium/src/+/a13b58bb24fb7bc0bbc7d68296ef11a64df4989e

[halliwell, 2017-03-23 13:29:35]

Description was changed from

==========
[Chromecast] Apply exclude_unwind_tables for appropriate builds

This reduces our installed APK size from ~90MB to 65 on ATV.  It
reduces earth OTA size by 1.5MB and earth cast_shell size by 5MB.

Leave unwind tables for builds where we regularly use a debugger (x86
and ATV debug).

BUG=internal b/36082628
==========

to

==========
[Chromecast] Apply exclude_unwind_tables for appropriate builds

This reduces our installed APK size from ~90MB to 65 on ATV.  It
reduces earth OTA size by 1.5MB and earth cast_shell size by 5MB.

Leave unwind tables for builds where we regularly use a debugger (x86
and ATV debug).

BUG=internal b/36082628
==========

[halliwell, 2017-03-23 13:29:36]

halliwell@chromium.org changed reviewers:
+ bcf@chromium.org, dpranke@chromium.org, slan@chromium.org

[bcf, 2017-03-23 18:27:22]

Personally, I use gdb on device quite a lot so I'm not sure 1.5MB is worth it.
It would be worth applying it to our release builds though.

[halliwell, 2017-03-23 18:45:22]

On 2017/03/23 18:27:22, bcf wrote:
> Personally, I use gdb on device quite a lot so I'm not sure 1.5MB is worth it.
> It would be worth applying it to our release builds though.

Heh, I had assumed low gdb usage (and figured it would be good to have OTA size
consistent across both) :)
Updated to exclude debug builds.

[slan, 2017-03-23 19:29:26]

LGTM, this matches what we're already doing with symbol_level.

[bcf, 2017-03-23 19:38:28]

lgtm

[halliwell, 2017-03-24 14:27:42]

On 2017/03/23 19:38:28, bcf wrote:
> lgtm

dpranke: PTAL.

[Dirk Pranke, 2017-03-24 21:16:20]

lgtm

[halliwell, 2017-03-24 21:47:17]

The CQ bit was checked by halliwell@chromium.org

[commit-bot: I haz the power, 2017-03-24 21:47:49]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-25 01:51:55]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490392037014920,
"parent_rev": "ad19ce4f06f1a12bf01780f85cef87e4e16fd8b3", "commit_rev":
"a13b58bb24fb7bc0bbc7d68296ef11a64df4989e"}

[commit-bot: I haz the power, 2017-03-25 01:52:47]

Description was changed from

==========
[Chromecast] Apply exclude_unwind_tables for appropriate builds

This reduces our installed APK size from ~90MB to 65 on ATV.  It
reduces earth OTA size by 1.5MB and earth cast_shell size by 5MB.

Leave unwind tables for builds where we regularly use a debugger (x86
and ATV debug).

BUG=internal b/36082628
==========

to

==========
[Chromecast] Apply exclude_unwind_tables for appropriate builds

This reduces our installed APK size from ~90MB to 65 on ATV.  It
reduces earth OTA size by 1.5MB and earth cast_shell size by 5MB.

Leave unwind tables for builds where we regularly use a debugger (x86
and ATV debug).

BUG=internal b/36082628

Review-Url: https://codereview.chromium.org/2770833003
Cr-Commit-Position: refs/heads/master@{#459619}
Committed:
https://chromium.googlesource.com/chromium/src/+/a13b58bb24fb7bc0bbc7d68296ef...
==========

[commit-bot: I haz the power, 2017-03-25 01:52:49]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/a13b58bb24fb7bc0bbc7d68296ef...