Remove min fps requirements for VDA unittest on windows bots.

Remove min fps requirements for VDA unittest on windows bots.

These bots can be intermittently slow, so requiring 50 fps isn't
possible. This patch specifies in the command-line argument
to the test that there's no minimum fps on the test file that's used.
The tests will still fail and time out if they take way too long.

BUG=680348
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2631783004
Cr-Commit-Position: refs/heads/master@{#450660}
Committed: https://chromium.googlesource.com/chromium/src/+/b3fdd063a1128b441a232215a5798416c3b63ef4

[jbauman, 2017-01-14 00:57:58]

Description was changed from

==========
Remove min fps requirements for VDA unittest on windows bots.

These bots are often overloaded with work, so requiring 50 fps isn't
possible (unlike on Chrome OS, where this is the only work done on the
machine at the time). This patch specifies in the command-line argument
to the test that there's no minimum fps on the test file that's used.
The tests will still fail and time out if they take way too long.

BUG=680348
==========

to

==========
Remove min fps requirements for VDA unittest on windows bots.

These bots are often overloaded with work, so requiring 50 fps isn't
possible (unlike on Chrome OS, where this is the only work done on the
machine at the time). This patch specifies in the command-line argument
to the test that there's no minimum fps on the test file that's used.
The tests will still fail and time out if they take way too long.

BUG=680348
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

[jbauman, 2017-01-14 00:58:04]

The CQ bit was checked by jbauman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-14 00:58:25]

Dry run: 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, 2017-01-14 03:06:30]

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

[commit-bot: I haz the power, 2017-01-14 03:06:31]

Dry run: This issue passed the CQ dry run.

[jbauman, 2017-01-17 22:33:48]

jbauman@chromium.org changed reviewers:
+ kbr@chromium.org

[Ken Russell (switch to Gerrit), 2017-01-17 23:51:46]

These bots run only one test at a time. The VDA unittests have full control of
the physical machine when they run. Given that, are you sure this is the reason
they are failing?

[jbauman, 2017-02-14 02:21:26]

Description was changed from

==========
Remove min fps requirements for VDA unittest on windows bots.

These bots are often overloaded with work, so requiring 50 fps isn't
possible (unlike on Chrome OS, where this is the only work done on the
machine at the time). This patch specifies in the command-line argument
to the test that there's no minimum fps on the test file that's used.
The tests will still fail and time out if they take way too long.

BUG=680348
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

to

==========
Remove min fps requirements for VDA unittest on windows bots.

These bots can be intermittently slow, so requiring 50 fps isn't
possible. This patch specifies in the command-line argument
to the test that there's no minimum fps on the test file that's used.
The tests will still fail and time out if they take way too long.

BUG=680348
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

[jbauman, 2017-02-14 02:21:53]

On 2017/01/17 23:51:46, Ken Russell wrote:
> These bots run only one test at a time. The VDA unittests have full control of
> the physical machine when they run. Given that, are you sure this is the
reason
> they are failing?

Ok, updated the commit message, though it just looks like the bots randomly get
slower.

[Ken Russell (switch to Gerrit), 2017-02-14 03:08:58]

lgtm

John, if you can help in diagnosing what might be happening here, we would
appreciate it. There is evidence of problems on some bots (
http://crbug.com/690765 ), but those bots didn't run the
video_decode_accelerator_unittest jobs which timed out.

[jbauman, 2017-02-15 00:36:07]

The CQ bit was checked by jbauman@chromium.org

[jbauman, 2017-02-15 00:36:12]

The CQ bit was unchecked by jbauman@chromium.org

[jbauman, 2017-02-15 01:20:27]

The CQ bit was checked by jbauman@chromium.org

[jbauman, 2017-02-15 01:20:28]

The patchset sent to the CQ was uploaded after l-g-t-m from kbr@chromium.org
Link to the patchset: https://codereview.chromium.org/2631783004/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2017-02-15 01:21:53]

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, 2017-02-15 03:58:23]

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

[commit-bot: I haz the power, 2017-02-15 03:58:24]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jbauman, 2017-02-15 10:24:41]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2017-02-15 10:25:03]

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, 2017-02-15 11:08:04]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487154281097360,
"parent_rev": "0bd4ea13dfe59ff723559061ed217b4f5a4cb4e1", "commit_rev":
"b3fdd063a1128b441a232215a5798416c3b63ef4"}

[commit-bot: I haz the power, 2017-02-15 11:08:39]

Description was changed from

==========
Remove min fps requirements for VDA unittest on windows bots.

These bots can be intermittently slow, so requiring 50 fps isn't
possible. This patch specifies in the command-line argument
to the test that there's no minimum fps on the test file that's used.
The tests will still fail and time out if they take way too long.

BUG=680348
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

to

==========
Remove min fps requirements for VDA unittest on windows bots.

These bots can be intermittently slow, so requiring 50 fps isn't
possible. This patch specifies in the command-line argument
to the test that there's no minimum fps on the test file that's used.
The tests will still fail and time out if they take way too long.

BUG=680348
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2631783004
Cr-Commit-Position: refs/heads/master@{#450660}
Committed:
https://chromium.googlesource.com/chromium/src/+/b3fdd063a1128b441a232215a579...
==========

[commit-bot: I haz the power, 2017-02-15 11:08:40]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/b3fdd063a1128b441a232215a579...