Use "-Os" for Android or IOS in "optimize_max" config

Use "-Os" for Android or IOS in "optimize_max" config

Using "-Os" would benefit both Android and IOS build, because "-Os" does
"-O2" optimizations and code size improvement.

libchrome.so drops from 44MB to 42 MB.
ChromePublic.apk drops from 47 MB to 45 MB.

BUG=621335
Committed: https://crrev.com/2bb3c4f7a7c7632afe1b63a19d49daa61bbfc37b
Cr-Commit-Position: refs/heads/master@{#406304}

[huapengl, 2016-07-18 17:41:00]

huapengl@amazon.com changed reviewers:
+ dpranke@chromium.org

[huapengl, 2016-07-18 17:41:01]

Hi,
Can you take a look at this CL?

Thanks

[Dirk Pranke, 2016-07-18 22:59:18]

lgtm. thanks for fixing this!

That's not so good, that "optimize_max" was actually worse than "optimize" in
this case.

sdefresne, agrieve, fyi ...

[agrieve, 2016-07-19 00:09:37]

On 2016/07/18 22:59:18, Dirk Pranke wrote:
> lgtm. thanks for fixing this!
> 
> That's not so good, that "optimize_max" was actually worse than "optimize" in
> this case.
> 
> sdefresne, agrieve, fyi ...

🙌

[sdefresne, 2016-07-19 08:25:12]

sdefresne@chromium.org changed reviewers:
+ sdefresne@chromium.org

[sdefresne, 2016-07-19 08:25:12]

Thank you!

[huapengl, 2016-07-19 16:02:21]

The CQ bit was checked by huapengl@amazon.com

[commit-bot: I haz the power, 2016-07-19 16:02:43]

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, 2016-07-19 17:19:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-19 17:21:55]

Description was changed from

==========
Use "-Os" for Android or IOS in "optimize_max" config

Using "-Os" would benefit both Android and IOS build, because "-Os" does
"-O2" optimizations and code size improvement.

libchrome.so drops from 44MB to 42 MB.
ChromePublic.apk drops from 47 MB to 45 MB.

BUG=621335
==========

to

==========
Use "-Os" for Android or IOS in "optimize_max" config

Using "-Os" would benefit both Android and IOS build, because "-Os" does
"-O2" optimizations and code size improvement.

libchrome.so drops from 44MB to 42 MB.
ChromePublic.apk drops from 47 MB to 45 MB.

BUG=621335

Committed: https://crrev.com/2bb3c4f7a7c7632afe1b63a19d49daa61bbfc37b
Cr-Commit-Position: refs/heads/master@{#406304}
==========

[commit-bot: I haz the power, 2016-07-19 17:21:56]

Patchset 1 (id:??) landed as
https://crrev.com/2bb3c4f7a7c7632afe1b63a19d49daa61bbfc37b
Cr-Commit-Position: refs/heads/master@{#406304}

[huapengl, 2016-07-20 15:41:44]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2164973002/ by huapengl@amazon.com.

The reason for reverting is: 8.1% regression in speedometer at 406285:406306

https://bugs.chromium.org/p/chromium/issues/detail?id=629798#c4.