commit f9871703e18cd8eb7b06a37733d94f05f647fbc1
author Brian Sheedy <bsheedy@chromium.org> Mon Mar 20 18:30:19 2023
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Mar 20 18:30:19 2023
tree 7851d0c99da015d2fc25e3c808eb7ee1a48526a0
parent ed6cfb88f978dcb09ff88fc3cac4b1cbaa343c62

Reland "Switch WebGL tests to heartbeat"

This reverts commit 85ae84b891629cd599a322f213c740cfe4cb0bfc.

Changes from original CL:
* None - Fuchsia issues were fixed by crrev.com/c/4298514 and
  crrev.com/c/4298549.

Original change's description:
> Revert "Switch WebGL tests to heartbeat"
>
> This reverts commit d88b9396508bc8901b436f1fbc90972b8d448d25.
>
> Reason for revert: Causing Fuchsia builders to time out
>
> Original change's description:
> > Switch WebGL tests to heartbeat
> >
> > Switches the WebGL conformance tests to use a heartbeat mechanism
> > like the WebGPU conformance tests instead of a fixed 5 minute
> > timeout. The 5 minute timeout still exists, but should basically
> > never be hit.
> >
> > The end result of this is that we should fail much faster in
> > cases of hung/crashed renderers. Before, we would take 5 minutes
> > to detect these, but now we can detect them within 15 seconds.
> > This will help normalize shard times and open up opportunities to
> > make further improvements such as lowering the shard timeout we
> > use.
> >
> > No noticeable impact to test runtime when tests run normally.
> >
> > Synopsis of changes under the hood:
> > * Moved the WebGL harness JavaScript code into dedicated
> >   JavaScript files instead of defining them directly in the Python
> >   test harness file.
> > * Generalized the WebGPU websocket code for use in both test
> >   suites
> > * Added Slow expectation support, which is necessary to address
> >   slow heartbeats on a few configurations
> > * Adjusted the number of parallel jobs on some configurations to
> >   ensure heartbeats are sent on time instead of being delayed due
> >   to the hardware being hit too hard
> >
> > Bug: 1354797
> > Change-Id: Ie862bd081d9e97947efd10400ab5c7337215124a
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3840756
> > Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
> > Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
> > Reviewed-by: Kenneth Russell <kbr@chromium.org>
> > Reviewed-by: Ben Pastene <bpastene@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1054455}
>
> Bug: 1354797
> Change-Id: Ia541bca2b4cd8955e6d8e3ea7adeb61f156626d6
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3926272
> Auto-Submit: Chong Gu <chonggu@google.com>
> Reviewed-by: Kenneth Russell <kbr@chromium.org>
> Reviewed-by: Ben Pastene <bpastene@chromium.org>
> Commit-Queue: Chong Gu <chonggu@google.com>
> Cr-Commit-Position: refs/heads/main@{#1055024}

Bug: 1354797
Change-Id: I8150188fc9d2b52e504d2c0cc2e4381d853f58b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4327254
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1119488}