1865251 - [wpt-sync] Sync PR 43223 - Revert "LoAF: count tasks with UI events towards blockingDuration"

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 43223 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/43223
Details from upstream follow.

Mingyu Lei <[e-mail]> wrote:

Revert "LoAF: count tasks with UI events towards blockingDuration"

This reverts commit dd63571a32b548d0a9ffe128da66f442080dc444.

Reason for revert: The WPT is failing, see: https://ci.chromium.org/ui/p/chromium/builders/ci/Linux%20Tests%20(dbg)(1)/117038/test-results

Original change's description:

LoAF: count tasks with UI events towards blockingDuration

Apparently UI events are sent in a task that's not associated
with a frame (using the input task source), so the
AnimationFrameTimingMonitor thought that it was a task
unrelated to the current widget.

Fixed by applying the task duration if this task produced
any UI events.

Bug: 1501836
Change-Id: Ib5827b29fc028be66f9ed98fb029d3638cd0848c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5033867
Reviewed-by: Michal Mocny \<[e-mail]>
Commit-Queue: Noam Rosenthal \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1225647}

Bug: 1501836
Change-Id: If875d4adaa7cabd4039292f621819c7ac1d8dcfe
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5040031
Owners-Override: Mingyu Lei \<[e-mail]>
Commit-Queue: Rubber Stamper \<[e-mail]>
Bot-Commit: Rubber Stamper \<[e-mail]>
Auto-Submit: Mingyu Lei \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1225886}

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

PR 43223 applied with additional changes from upstream: c714216ba4ef5cf3e5328460260fa5538047ead0

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=dd32f09ff00a1dd98c57b9dfee65a86f6027bf2e

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ba916de019c8
[wpt PR 43223] - Revert "LoAF: count tasks with UI events towards blockingDuration", a=testonly

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7b2c115e9da9
[wpt PR 43223] - Revert "LoAF: count tasks with UI events towards blockingDuration", a=testonly

Comment 7

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/7b2c115e9da9