[BotHealth] Disable PCv2 on reference using ShouldDisable

[BotHealth] Disable PCv2 on reference using ShouldDisable

As of crrev.com/2249623002, super-/subclass decorator usages no longer
affect sub-/superclasses respectively. PCv2 is currently intended to be
disabled on all reference builds until the ref build is rolled, and this had
previously been done using the @Disabled decorator on the superclass.

This CL ports that functionality into PCv2's ShouldDisable method.

BUG=619254
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Committed: https://crrev.com/fc7e75b896b8b0905a295300d9482af562e023e9
Cr-Commit-Position: refs/heads/master@{#412400}

[eakuefner, 2016-08-16 17:33:22]

Description was changed from

==========
[BotHealth] Disable PCv2 on reference using ShouldDisable

As of crrev.com/2249623002, super-/subclass decorator usages no longer
affect sub-/superclasses respectively. PCv2 is currently intended to be
disabled on all reference builds until the ref build is rolled, and this had
previously been done using the @Disabled decorator on the superclass.

This CL ports that functionality into PCv2's ShouldDisable method.

BUG=619254
==========

to

==========
[BotHealth] Disable PCv2 on reference using ShouldDisable

As of crrev.com/2249623002, super-/subclass decorator usages no longer
affect sub-/superclasses respectively. PCv2 is currently intended to be
disabled on all reference builds until the ref build is rolled, and this had
previously been done using the @Disabled decorator on the superclass.

This CL ports that functionality into PCv2's ShouldDisable method.

BUG=619254
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========

[eakuefner, 2016-08-16 17:33:47]

eakuefner@chromium.org changed reviewers:
+ aiolos@chromium.org, nednguyen@google.com

[eakuefner, 2016-08-16 17:33:47]

PTAL. I have tested locally to ensure that this solution works.

[aiolos (Not reviewing), 2016-08-16 17:35:04]

lgtm

[nednguyen, 2016-08-16 17:35:04]

lgtm

[eakuefner, 2016-08-16 17:35:52]

The CQ bit was checked by eakuefner@chromium.org

[commit-bot: I haz the power, 2016-08-16 17:36:04]

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-08-16 21:34:35]

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

[commit-bot: I haz the power, 2016-08-16 21:34:36]

Try jobs failed on following builders:
  winx64_10_perf_cq on master.tryserver.chromium.perf (JOB_TIMED_OUT, no build
URL)

[eakuefner, 2016-08-16 23:09:15]

The CQ bit was checked by eakuefner@chromium.org

[commit-bot: I haz the power, 2016-08-16 23:09:58]

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-08-17 00:41:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-17 00:44:30]

Description was changed from

==========
[BotHealth] Disable PCv2 on reference using ShouldDisable

As of crrev.com/2249623002, super-/subclass decorator usages no longer
affect sub-/superclasses respectively. PCv2 is currently intended to be
disabled on all reference builds until the ref build is rolled, and this had
previously been done using the @Disabled decorator on the superclass.

This CL ports that functionality into PCv2's ShouldDisable method.

BUG=619254
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========

to

==========
[BotHealth] Disable PCv2 on reference using ShouldDisable

As of crrev.com/2249623002, super-/subclass decorator usages no longer
affect sub-/superclasses respectively. PCv2 is currently intended to be
disabled on all reference builds until the ref build is rolled, and this had
previously been done using the @Disabled decorator on the superclass.

This CL ports that functionality into PCv2's ShouldDisable method.

BUG=619254
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

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

[commit-bot: I haz the power, 2016-08-17 00:44:31]

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