Switched remaining GPU test bots to be pure testers, triggered by builders.

Switched remaining GPU test bots to be pure testers, triggered by builders.

Note that the try servers on the tryserver.chromium waterfall remain
independent builder/testers.

Tested locally with runbuild.py, masters_test.py, and masters_cfg_test.py.

BUG=310926
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=235483

[Ken Russell (switch to Gerrit), 2013-11-16 00:29:14]

Robbie: please review. Thanks.

Mike, Aaron: FYI.

[Ken Russell (switch to Gerrit), 2013-11-16 00:39:52]

+bajones, zmo as FYI

[ghost stip (do not use), 2013-11-16 01:15:28]

some comments

https://chromiumcodereview.appspot.com/74063005/diff/1/masters/master.chromiu...
File masters/master.chromium.gpu/slaves.cfg (right):

https://chromiumcodereview.appspot.com/74063005/diff/1/masters/master.chromiu...
masters/master.chromium.gpu/slaves.cfg:211: 'version': 'precise',
trailing space?

https://chromiumcodereview.appspot.com/74063005/diff/1/masters/master.chromiu...
File masters/master.chromium.webkit/master_gpu_latest_cfg.py (right):

https://chromiumcodereview.appspot.com/74063005/diff/1/masters/master.chromiu...
masters/master.chromium.webkit/master_gpu_latest_cfg.py:136:
polling_builders.append(builder)
don't need polling_builders

https://chromiumcodereview.appspot.com/74063005/diff/1/masters/master.chromiu...
masters/master.chromium.webkit/master_gpu_latest_cfg.py:159:
factory_properties))
add trigger if builder_name in trigger_name_map

[Ken Russell (switch to Gerrit), 2013-11-16 01:25:05]

https://codereview.chromium.org/74063005/diff/1/masters/master.chromium.gpu/s...
File masters/master.chromium.gpu/slaves.cfg (right):

https://codereview.chromium.org/74063005/diff/1/masters/master.chromium.gpu/s...
masters/master.chromium.gpu/slaves.cfg:211: 'version': 'precise',
On 2013/11/16 01:15:28, stip wrote:
> trailing space?

Fixed.

https://codereview.chromium.org/74063005/diff/1/masters/master.chromium.webki...
File masters/master.chromium.webkit/master_gpu_latest_cfg.py (right):

https://codereview.chromium.org/74063005/diff/1/masters/master.chromium.webki...
masters/master.chromium.webkit/master_gpu_latest_cfg.py:136:
polling_builders.append(builder)
On 2013/11/16 01:15:28, stip wrote:
> don't need polling_builders

Done.

https://codereview.chromium.org/74063005/diff/1/masters/master.chromium.webki...
masters/master.chromium.webkit/master_gpu_latest_cfg.py:159:
factory_properties))
On 2013/11/16 01:15:28, stip wrote:
> add trigger if builder_name in trigger_name_map

Thanks for catching that. That would have been a bad bug.

[ghost stip (do not use), 2013-11-16 01:27:23]

lgtm

[commit-bot: I haz the power, 2013-11-16 01:27:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kbr@chromium.org/74063005/80001

[commit-bot: I haz the power, 2013-11-16 01:28:32]

Change committed as 235483