[wpt-sync] Sync PR 36915 - Address flakiness issue in permissions/permissions-cg.https.html
Categories
(Testing :: web-platform-tests, task, P4)
Tracking
(firefox108 fixed)
Tracking | Status | |
---|---|---|
firefox108 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 36915 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/36915
Details from upstream follow.
Chris Dumez <cdumez@apple.com> wrote:
Address flakiness issue in permissions/permissions-cg.https.html
Make sure the PermissionStatus object gets constructed and its
change
event listener gets registered BEFORE we update the permission state. Previously, the order wasn't guaranteed and this would lead to flakiness because thechange
would get fired before the event listener got added.Also trigger garbage collection explicitly to make it more likely to trigger GC bugs, given that this is what the test intends to cover.
This is a merge from https://github.com/WebKit/WebKit/pull/6358.
Assignee | ||
Comment 1•2 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=df95041cf08e57796bbde49d074adac76bb7dec7
Assignee | ||
Comment 2•2 years ago
|
||
CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 1 tests and 1 subtests
Status Summary
Firefox
TIMEOUT
: 1
ERROR
: 1
Chrome
ERROR
: 1
Safari
TIMEOUT
: 1
ERROR
: 1
Links
Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base
Details
New Tests That Don't Pass
- /permissions/permissions-cg.https.html [wpt.fyi]:
ERROR
(Chrome:ERROR
, Safari:ERROR
)- status is not garbage collected when it goes out of scope:
TIMEOUT
(Safari:TIMEOUT
)
- status is not garbage collected when it goes out of scope:
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8b37a3482329 [wpt PR 36915] - Address flakiness issue in permissions/permissions-cg.https.html, a=testonly https://hg.mozilla.org/integration/autoland/rev/2c8a33fda88d [wpt PR 36915] - Update wpt metadata, a=testonly
Comment 4•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8b37a3482329
https://hg.mozilla.org/mozilla-central/rev/2c8a33fda88d
Description
•