[build] Use same symbol level as chromium for android

[build] Use same symbol level as chromium for android

This makes our configuration similar to Chromium's for
android performance testing.

This blocks deps'ing in:
https://codereview.chromium.org/2377013002

BUG=chromium:648948
NOTRY=true

Committed: https://crrev.com/679409e800a173ef7830ba8781dfa83506c59ccf
Cr-Commit-Position: refs/heads/master@{#39915}

[Michael Achenbach, 2016-09-30 11:19:36]

Description was changed from

==========
[build] Use same symbol level as chromium for android

BUG=
==========

to

==========
[build] Use same symbol level as chromium for android

This makes our configuration similar to Chromium's for
android performance testing.

This blocks deps'ing in:
https://codereview.chromium.org/2377013002

BUG=chromium:648948
==========

[Michael Achenbach, 2016-09-30 11:19:36]

machenbach@chromium.org changed reviewers:
+ hablich@chromium.org, jochen@chromium.org, vogelheim@chromium.org

[Michael Achenbach, 2016-09-30 11:19:59]

PTAL

[vogelheim, 2016-09-30 11:27:59]

I don't get it...

CL description says, "Use same symbol level as chromium for android", but this
introduces "minimal_symbols" mixin to gn_release_android_arm*, but
gn_release_arm doesn't have it. Wouldn't this CL then make those configs
different? I guess I'm misunderstanding something?

[Michael Achenbach, 2016-09-30 11:43:52]

On 2016/09/30 11:27:59, vogelheim wrote:
> I don't get it...
> 
> CL description says, "Use same symbol level as chromium for android", but this
> introduces "minimal_symbols" mixin to gn_release_android_arm*, but
> gn_release_arm doesn't have it. Wouldn't this CL then make those configs
> different? I guess I'm misunderstanding something?

Arm for chromebooks and Android arm are separate configs. We will probably never
run into the 4GB limit anyways as stated in http://crbug.com/648948.

This is to not trigger an assert added in
https://codereview.chromium.org/2377013002 for android.

This makes sure we build android in the same way as it's built for chromium
perf:
https://cs.chromium.org/chromium/src/tools/mb/mb_config.pyl?q=mb_config&sq=pa...

[vogelheim, 2016-09-30 11:46:05]

lgtm

[Michael Achenbach, 2016-09-30 11:47:50]

Description was changed from

==========
[build] Use same symbol level as chromium for android

This makes our configuration similar to Chromium's for
android performance testing.

This blocks deps'ing in:
https://codereview.chromium.org/2377013002

BUG=chromium:648948
==========

to

==========
[build] Use same symbol level as chromium for android

This makes our configuration similar to Chromium's for
android performance testing.

This blocks deps'ing in:
https://codereview.chromium.org/2377013002

BUG=chromium:648948
NOTRY=true
==========

[Michael Achenbach, 2016-09-30 11:56:18]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-09-30 11:56:27]

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-09-30 11:59:01]

Description was changed from

==========
[build] Use same symbol level as chromium for android

This makes our configuration similar to Chromium's for
android performance testing.

This blocks deps'ing in:
https://codereview.chromium.org/2377013002

BUG=chromium:648948
NOTRY=true
==========

to

==========
[build] Use same symbol level as chromium for android

This makes our configuration similar to Chromium's for
android performance testing.

This blocks deps'ing in:
https://codereview.chromium.org/2377013002

BUG=chromium:648948
NOTRY=true
==========

[commit-bot: I haz the power, 2016-09-30 11:59:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-30 11:59:18]

Description was changed from

==========
[build] Use same symbol level as chromium for android

This makes our configuration similar to Chromium's for
android performance testing.

This blocks deps'ing in:
https://codereview.chromium.org/2377013002

BUG=chromium:648948
NOTRY=true
==========

to

==========
[build] Use same symbol level as chromium for android

This makes our configuration similar to Chromium's for
android performance testing.

This blocks deps'ing in:
https://codereview.chromium.org/2377013002

BUG=chromium:648948
NOTRY=true

Committed: https://crrev.com/679409e800a173ef7830ba8781dfa83506c59ccf
Cr-Commit-Position: refs/heads/master@{#39915}
==========

[commit-bot: I haz the power, 2016-09-30 11:59:19]

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