Removed references to mac_gpu, which has been removed

Removed references to mac_gpu, which has been removed

BUG=426897
R=kbr@chromium.org,phajdan.jr@chromium.org,zmo@chromium.org

Committed: https://crrev.com/2adbe15c19c12ed068924813f98621a08326441e
Cr-Commit-Position: refs/heads/master@{#308995}

[Paweł Hajdan Jr., 2014-12-12 17:45:06]

LGTM

[Zhenyao Mo, 2014-12-12 17:49:20]

LGTM

[Ken Russell (switch to Gerrit), 2014-12-12 18:47:47]

LGTM with this question answered.

https://codereview.chromium.org/805483002/diff/1/PRESUBMIT.py
File PRESUBMIT.py (right):

https://codereview.chromium.org/805483002/diff/1/PRESUBMIT.py#newcode1706
PRESUBMIT.py:1706: 'mac_chromium_rel_ng': ['defaulttests'],
Same question as for the CL removing linux_gpu and win_gpu: will
mac_chromium_rel_ng unconditionally run the GPU tests, or are the tests that are
run filtered by the "standard_tests" list above?

[Sergiy Byelozyorov, 2014-12-15 16:00:23]

https://codereview.chromium.org/805483002/diff/1/PRESUBMIT.py
File PRESUBMIT.py (right):

https://codereview.chromium.org/805483002/diff/1/PRESUBMIT.py#newcode1706
PRESUBMIT.py:1706: 'mac_chromium_rel_ng': ['defaulttests'],
On 2014/12/12 18:47:47, Ken Russell wrote:
> Same question as for the CL removing linux_gpu and win_gpu: will
> mac_chromium_rel_ng unconditionally run the GPU tests, or are the tests that
are
> run filtered by the "standard_tests" list above?

I think 'defaulttests' is just a placeholder, which is not used anywhere. Pawel,
WDYT?

[Paweł Hajdan Jr., 2014-12-15 16:01:36]

https://codereview.chromium.org/805483002/diff/1/PRESUBMIT.py
File PRESUBMIT.py (right):

https://codereview.chromium.org/805483002/diff/1/PRESUBMIT.py#newcode1706
PRESUBMIT.py:1706: 'mac_chromium_rel_ng': ['defaulttests'],
On 2014/12/15 16:00:23, Sergiy Byelozyorov wrote:
> On 2014/12/12 18:47:47, Ken Russell wrote:
> > Same question as for the CL removing linux_gpu and win_gpu: will
> > mac_chromium_rel_ng unconditionally run the GPU tests, or are the tests that
> are
> > run filtered by the "standard_tests" list above?
> 
> I think 'defaulttests' is just a placeholder, which is not used anywhere.
Pawel,
> WDYT?

Yeah, this is just a dummy value at this point. testfilter has no effect on
recipe-based bots.

[Sergiy Byelozyorov, 2014-12-15 16:02:41]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2014-12-15 16:02:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/805483002/1

[commit-bot: I haz the power, 2014-12-15 16:07:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-15 16:07:27]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Ken Russell (switch to Gerrit), 2014-12-15 19:07:36]

kbr@chromium.org changed reviewers:
+ piman@chromium.org, sievers@chromium.org

[Ken Russell (switch to Gerrit), 2014-12-15 19:07:36]

piman, sievers: OWNERS review of gpu/PRESUBMIT.py please.

[piman, 2014-12-15 19:09:20]

lgtm

[Ken Russell (switch to Gerrit), 2014-12-15 19:09:40]

The CQ bit was checked by kbr@chromium.org

[commit-bot: I haz the power, 2014-12-15 19:11:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/805483002/1

[commit-bot: I haz the power, 2014-12-15 19:18:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-15 19:18:48]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Sergiy Byelozyorov, 2014-12-18 11:02:32]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2014-12-18 11:03:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/805483002/1

[commit-bot: I haz the power, 2014-12-18 11:08:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-18 11:08:01]

Try jobs failed on following builders:
  android_compile_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_compile...)

[Sergiy Byelozyorov, 2014-12-18 11:25:47]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2014-12-18 11:26:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/805483002/20001

[commit-bot: I haz the power, 2014-12-18 12:21:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-12-18 12:22:14]

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