Improve compile time for 'Blimp Client Engine Integration' bot

Improve compile time for 'Blimp Client Engine Integration' bot

Add 'use_goma = true' for gn args.

BUG=656837
Committed: https://crrev.com/6b9fe5c1b2e37c4a6cb9088f23377162b7fd2924
Cr-Commit-Position: refs/heads/master@{#426027}

[shenghuazhang, 2016-10-18 00:52:10]

shenghuazhang@chromium.org changed reviewers:
+ dpranke@chromium.org, mikecase@chromium.org

[mikecase (-- gone --), 2016-10-18 01:05:48]

I would name the files like...


build/args/bots/chromium.fyi/blimp_client_bot_rel.gn

... if you take this approach.

Alternatively you could,

1) Add support to mb.py script so that you can specify configs in .gn files AND
add some additional configs to those

2) Just explicitly list out all the GN args in the mb_config.pyl file instead of
using any of these .gn files.

[mikecase (-- gone --), 2016-10-18 01:09:25]

wait, maybe just add...

use_goma = true

...to blimp_client_rel.gn

I just noticed these configs are expliticly for the bots. Im not sure why these
configs don't have use_goma already.

[shenghuazhang, 2016-10-18 01:25:30]

Thanks for the reconsidering! I modify the two gn files simply in this patch.

[mikecase (-- gone --), 2016-10-18 01:36:22]

non-OWNER lgtm

[Dirk Pranke, 2016-10-18 01:52:29]

lgtm

[shenghuazhang, 2016-10-18 18:54:22]

The CQ bit was checked by shenghuazhang@chromium.org

[commit-bot: I haz the power, 2016-10-18 18:54:35]

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-10-18 19:55:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-21 13:02:25]

Description was changed from

==========
Improve compile time for 'Blimp Client Engine Integration' bot

Add 'use_goma = true' for gn args.

BUG=656837
==========

to

==========
Improve compile time for 'Blimp Client Engine Integration' bot

Add 'use_goma = true' for gn args.

BUG=656837

Committed: https://crrev.com/6b9fe5c1b2e37c4a6cb9088f23377162b7fd2924
Cr-Commit-Position: refs/heads/master@{#426027}
==========

[commit-bot: I haz the power, 2016-10-21 13:02:26]

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