[Android] Add configurations for cronet continuous builder + trybot.

[Android] Add configurations for cronet continuous builder + trybot.

BUG=634350
Committed: https://crrev.com/64e96a0f101d325646f3759b676dc422612c7fd8
Cr-Commit-Position: refs/heads/master@{#416989}

[jbudorick, 2016-09-02 18:05:50]

jbudorick@chromium.org changed reviewers:
+ stip@chromium.org, xunjieli@chromium.org

[xunjieli, 2016-09-02 18:21:03]

https://codereview.chromium.org/2302353003/diff/1/testing/buildbot/chromium.a...
File testing/buildbot/chromium.android.json (right):

https://codereview.chromium.org/2302353003/diff/1/testing/buildbot/chromium.a...
testing/buildbot/chromium.android.json:4: "cronet"
Can we compile "cronet_package" target? which will compile all dependent targets
like :cronet, :cronet_api, etc.

https://codereview.chromium.org/2302353003/diff/1/tools/mb/mb_config.pyl
File tools/mb/mb_config.pyl (right):

https://codereview.chromium.org/2302353003/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:1105: 'android', 'cronet', 'gn', 'release_trybot'
Do we need 'arm_no_neon' here? What architecture will the trybot run on? If it's
armv7, then we need 'arm_no_neon' so the cronet_package target can compile?

[jbudorick, 2016-09-02 18:27:19]

https://codereview.chromium.org/2302353003/diff/1/testing/buildbot/chromium.a...
File testing/buildbot/chromium.android.json (right):

https://codereview.chromium.org/2302353003/diff/1/testing/buildbot/chromium.a...
testing/buildbot/chromium.android.json:4: "cronet"
On 2016/09/02 18:21:02, xunjieli wrote:
> Can we compile "cronet_package" target? which will compile all dependent
targets
> like :cronet, :cronet_api, etc.

ah, yeah. Done.

https://codereview.chromium.org/2302353003/diff/1/tools/mb/mb_config.pyl
File tools/mb/mb_config.pyl (right):

https://codereview.chromium.org/2302353003/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:1105: 'android', 'cronet', 'gn', 'release_trybot'
On 2016/09/02 18:21:02, xunjieli wrote:
> Do we need 'arm_no_neon' here? What architecture will the trybot run on? If
it's
> armv7, then we need 'arm_no_neon' so the cronet_package target can compile?

For now, the build products from this bot will not be used at all, but this
configuration is for armv7. arm_no_neon added.

[xunjieli, 2016-09-02 18:30:37]

lgtm! Thanks a lot for adding the CQ support!

[ghost stip (do not use), 2016-09-02 18:32:07]

lgtm

[jbudorick, 2016-09-02 18:32:42]

jbudorick@chromium.org changed reviewers:
+ dpranke@chromium.org

[jbudorick, 2016-09-02 18:32:42]

+dpranke for owners

[jbudorick, 2016-09-02 18:32:56]

On 2016/09/02 18:32:42, jbudorick wrote:
> +dpranke for owners

(tools/mb/ owners, that is)

[Dirk Pranke, 2016-09-02 20:24:52]

https://codereview.chromium.org/2302353003/diff/20001/tools/mb/mb_config.pyl
File tools/mb/mb_config.pyl (right):

https://codereview.chromium.org/2302353003/diff/20001/tools/mb/mb_config.pyl#...
tools/mb/mb_config.pyl:1105: 'android', 'cronet', 'gn', 'release_trybot',
'arm_no_neon'
This should have 'arm_no_neon' in the config name, to make it clearer that it's
special.

[jbudorick, 2016-09-02 22:46:18]

https://codereview.chromium.org/2302353003/diff/20001/tools/mb/mb_config.pyl
File tools/mb/mb_config.pyl (right):

https://codereview.chromium.org/2302353003/diff/20001/tools/mb/mb_config.pyl#...
tools/mb/mb_config.pyl:1105: 'android', 'cronet', 'gn', 'release_trybot',
'arm_no_neon'
On 2016/09/02 20:24:52, Dirk Pranke wrote:
> This should have 'arm_no_neon' in the config name, to make it clearer that
it's
> special.
> 

Done & modified the other uses of arm_no_neon similarly.

[Dirk Pranke, 2016-09-06 01:01:42]

lgtm, thanks.

[jbudorick, 2016-09-07 15:56:25]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-09-07 15:56:26]

The patchset sent to the CQ was uploaded after l-g-t-m from stip@chromium.org,
xunjieli@chromium.org, dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/2302353003/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2016-09-07 15:56:44]

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-07 17:50:00]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-09-07 17:52:31]

Description was changed from

==========
[Android] Add configurations for cronet continuous builder + trybot.

BUG=634350
==========

to

==========
[Android] Add configurations for cronet continuous builder + trybot.

BUG=634350

Committed: https://crrev.com/64e96a0f101d325646f3759b676dc422612c7fd8
Cr-Commit-Position: refs/heads/master@{#416989}
==========

[commit-bot: I haz the power, 2016-09-07 17:52:32]

Patchset 4 (id:??) landed as
https://crrev.com/64e96a0f101d325646f3759b676dc422612c7fd8
Cr-Commit-Position: refs/heads/master@{#416989}