Add a mb config to android_cronet_tester trybot

Add a mb config to android_cronet_tester trybot

BUG=597596
Committed: https://crrev.com/2a56b36b8744bdf0ce95eadce89e14e55c5af971
Cr-Commit-Position: refs/heads/master@{#383099}

[xunjieli, 2016-03-24 14:08:30]

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

[xunjieli, 2016-03-24 14:08:31]

PTAL.

[jbudorick, 2016-03-24 15:24:20]

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

[jbudorick, 2016-03-24 15:24:21]

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

https://codereview.chromium.org/1832843002/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:649: 'android_cronet_tester':
'android_cronet_gyp_debug_trybot',
nit: alphabetize

https://codereview.chromium.org/1832843002/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:852: 'android_cronet_gyp_debug_trybot': [
nit: alphabetize

[xunjieli, 2016-03-24 15:56:43]

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

https://codereview.chromium.org/1832843002/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:649: 'android_cronet_tester':
'android_cronet_gyp_debug_trybot',
On 2016/03/24 15:24:21, jbudorick wrote:
> nit: alphabetize

Done.

https://codereview.chromium.org/1832843002/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:852: 'android_cronet_gyp_debug_trybot': [
On 2016/03/24 15:24:20, jbudorick wrote:
> nit: alphabetize

Done.

[Dirk Pranke, 2016-03-24 16:07:11]

lgtm w/ one change.

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

https://codereview.chromium.org/1832843002/diff/20001/tools/mb/mb_config.pyl#...
tools/mb/mb_config.pyl:870: 'android', 'cronet', 'gyp', 'debug_trybot', 'arm'
The 'arm' config is unnecessary, since target_os="android" means that arm is the
default, and we try to omit default configs in MB.

[xunjieli, 2016-03-24 16:20:02]

Thanks!

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

https://codereview.chromium.org/1832843002/diff/20001/tools/mb/mb_config.pyl#...
tools/mb/mb_config.pyl:870: 'android', 'cronet', 'gyp', 'debug_trybot', 'arm'
On 2016/03/24 16:07:11, Dirk Pranke wrote:
> The 'arm' config is unnecessary, since target_os="android" means that arm is
the
> default, and we try to omit default configs in MB.

Done.

[xunjieli, 2016-03-24 16:34:36]

The CQ bit was checked by xunjieli@chromium.org

[xunjieli, 2016-03-24 16:34:36]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/1832843002/#ps40001
(title: "Omit arm")

[commit-bot: I haz the power, 2016-03-24 16:34:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1832843002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1832843002/40001

[commit-bot: I haz the power, 2016-03-24 18:15:33]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-24 18:17:10]

Description was changed from

==========
Add a mb config to android_cronet_tester trybot

BUG=597596
==========

to

==========
Add a mb config to android_cronet_tester trybot

BUG=597596

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

[commit-bot: I haz the power, 2016-03-24 18:17:11]

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