1957608 - Perma /scheduler/tentative/yield/yield-priority-timers.any.worker.html | yield() with timer tasks (inherit signal) - assert_equals: expected "t1,y1,y2,y3,t2,t3" but got "t1,t2,t3,y1,y2,y3" - when Gecko 139 merges to beta on 2025-04-28

Status: VERIFIED FIXED

(Testing :: web-platform-tests, defect)

Description

[amarc]

Central-as-beta-simulation

How to run these simulations

Failure log

• Failure snippet:

[task 2025-04-01T11:12:42.592Z] 11:12:42     INFO - TEST-START | /scheduler/tentative/yield/yield-priority-timers.any.worker.html
[task 2025-04-01T11:12:42.644Z] 11:12:42     INFO - Closing window f41575c0-a8ab-4017-bb2c-16d9d578da78
[task 2025-04-01T11:12:44.351Z] 11:12:44     INFO - 
[task 2025-04-01T11:12:44.351Z] 11:12:44     INFO - TEST-UNEXPECTED-FAIL | /scheduler/tentative/yield/yield-priority-timers.any.worker.html | yield() with timer tasks (inherit signal) - assert_equals: expected "t1,y1,y2,y3,t2,t3" but got "t1,t2,t3,y1,y2,y3"
[task 2025-04-01T11:12:44.351Z] 11:12:44     INFO - @http://web-platform.test:8000/scheduler/tentative/yield/yield-priority-timers.any.js:36:16
[task 2025-04-01T11:12:44.351Z] 11:12:44     INFO - TEST-OK | /scheduler/tentative/yield/yield-priority-timers.any.worker.html | took 1759ms
[task 2025-04-01T11:12:44.357Z] 11:12:44     INFO - PID 28584 | 1743505964356	Marionette	INFO	Stopped listening on port 42923
[task 2025-04-01T11:12:48.834Z] 11:12:48     INFO - Browser exited with return code -15
[task 2025-04-01T11:12:48.835Z] 11:12:48     INFO - Closing logging queue
[task 2025-04-01T11:12:48.836Z] 11:12:48     INFO - queue closed
[task 2025-04-01T11:12:48.846Z] 11:12:48     INFO - INFO | runtests.py | TSan using symbolizer at /builds/worker/workspace/build/application/firefox/llvm-symbolizer
[task 2025-04-01T11:12:48.847Z] 11:12:48     INFO - Application command: /builds/worker/workspace/build/application/firefox/firefox --marionette --remote-allow-system-access about:blank -profile /tmp/tmphqxm6ih2
[task 2025-04-01T11:12:48.864Z] 11:12:48     INFO - PID 29234 | ATTENTION: default value of option mesa_glthread overridden by environment.
[task 2025-04-01T11:12:48.865Z] 11:12:48     INFO - PID 29234 | 1743505950888	Marionette	INFO	Marionette enabled
[task 2025-04-01T11:12:48.869Z] 11:12:48     INFO - PID 29234 | ATTENTION: default value of option mesa_glthread overridden by environment.
[task 2025-04-01T11:12:48.870Z] 11:12:48     INFO - PID 29234 | 1743505952705	Marionette	INFO	Listening on port 59950
[task 2025-04-01T11:12:48.873Z] 11:12:48     INFO - PID 29234 | Crash Annotation GraphicsCriticalError: |[0][GFX1-]: Calling WaitFlushedEvent::Run: is delayed: 4421 (t=18.3732) [GFX1-]: Calling WaitFlushedEvent::Run: is delayed: 4421
[task 2025-04-01T11:12:48.874Z] 11:12:48     INFO - Starting runner
[task 2025-04-01T11:12:53.264Z] 11:12:53     INFO - TEST-START | /scheduler/tentative/yield/yield-scripted-subframe-propagation.html

Comment 1

[Sean Feng [:sefeng]]

Attachment: Bug 1957608 - Fix yield-priority-timers.any.worker.html for beta merge

This test depends on TimeoutManager to run in workers, not sure
when this will be shipped, so flipping the pref for this test
to make it pass.

Comment 2

[amarc]

After applying your patch on the latest central as beta simulation, the issue is still present as you can see here + your patch caused this failure

Comment 3

[Sean Feng [:sefeng]]

Still looking

Comment 4

[Sean Feng [:sefeng]]

I have queued the patch to land, it should fix it.

Try: https://treeherder.mozilla.org/jobs?repo=try&revision=272ed7caab7c1ce212a27e65a25f57c56a1058a5

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e8ce51f3c98b
Fix yield-priority-timers.any.worker.html for beta merge r=smaug

Comment 6

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/e8ce51f3c98b

Comment 7

[amarc]

This failure still present: https://treeherder.mozilla.org/logviewer?job_id=502200094&repo=try&lineNumber=4164

Comment 8

[Sean Feng [:sefeng]]

Ah this is landed https://bugzilla.mozilla.org/show_bug.cgi?id=1954770, so the test is now perma pass.

I think we can just remove the landed patch in this bug, it should work now.

Can I request to backout the landed patch in this bug?

Comment 9

[amarc]

Yes, will back it out!

Comment 10

[amarc]

Backed out changeset e8ce51f3c98b (Bug 1957608) by dev request: https://hg.mozilla.org/integration/autoland/rev/b6b08ac6ef0c5b15fdb8d49cf8a0a9e113b3796c

Comment 11

[amarc]

Verified fixed in today's central as beta simulation