Add mb config for chromium.chromedriver.

Add mb config for chromium.chromedriver.

BUG=chromedriver:1384
Committed: https://crrev.com/a2ff7b4f84bc168eb6f5fc2b5758cabf49876184
Cr-Commit-Position: refs/heads/master@{#407065}

[samuong, 2016-07-21 23:33:00]

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

[samuong, 2016-07-21 23:33:00]

John, does this look right to you?

samuong@samuong1:~/chromium/src$ tools/mb/mb.py gen -b Linux -m
chromium.chromedriver -nv out/Default
out/Default/mb_type missing, clobbering to be safe

Writing """\
is_component_build = false
is_debug = false
use_goma = true
""" to /usr/local/google/home/samuong/ssd/chromium/src/out/Default/args.gn.

/usr/local/google/home/samuong/ssd/chromium/src/buildtools/linux64/gn gen
out/Default --check

[jbudorick, 2016-07-22 00:47:31]

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

https://codereview.chromium.org/2170163002/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:1144: 'gn_goma_release_static': [
I think these are equivalent to gn_release_bot and gn_release_bot_x86.

[samuong, 2016-07-22 02:08:46]

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

https://codereview.chromium.org/2170163002/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:1144: 'gn_goma_release_static': [
On 2016/07/22 00:47:31, jbudorick wrote:
> I think these are equivalent to gn_release_bot and gn_release_bot_x86.

Done.

[jbudorick, 2016-07-22 02:19:44]

lgtm, not an owner

[samuong, 2016-07-22 02:50:33]

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

[samuong, 2016-07-22 02:50:34]

This is the first of two CLs, which convert the ChromeDriver bots over to GN.
This CL adds chromedriver to the meta build config, which is used by the second
CL at https://codereview.chromium.org/2171093002/. PTAL?

[Dirk Pranke, 2016-07-22 03:11:46]

lgtm w/ a question ...

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

https://codereview.chromium.org/2170163002/diff/20001/tools/mb/mb_config.pyl#...
tools/mb/mb_config.pyl:76: 'Linux32': 'gn_release_bot_x86',
Do you need Linux 32? We don't support Chrome on Linux 32 anymore ...

[samuong, 2016-07-22 03:46:27]

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

https://codereview.chromium.org/2170163002/diff/20001/tools/mb/mb_config.pyl#...
tools/mb/mb_config.pyl:76: 'Linux32': 'gn_release_bot_x86',
On 2016/07/22 03:11:46, Dirk Pranke wrote:
> Do you need Linux 32? We don't support Chrome on Linux 32 anymore ...

We still have a linux32 bot, which tests chromedriver against 32-bit chromium
builds. It'll probably go away one day, but in the meantime it's still there, so
it needs to be configured.

[samuong, 2016-07-22 03:46:31]

The CQ bit was checked by samuong@chromium.org

[commit-bot: I haz the power, 2016-07-22 03:47:07]

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-22 05:12:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-22 05:14:25]

Description was changed from

==========
Add mb config for chromium.chromedriver.

BUG=chromedriver:1384
==========

to

==========
Add mb config for chromium.chromedriver.

BUG=chromedriver:1384

Committed: https://crrev.com/a2ff7b4f84bc168eb6f5fc2b5758cabf49876184
Cr-Commit-Position: refs/heads/master@{#407065}
==========

[commit-bot: I haz the power, 2016-07-22 05:14:25]

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