github web-platform-tests/wpt merge_pr_47273

latest releases: merge_pr_49470, epochs/three_hourly/2024-12-03_06H, epochs/six_hourly/2024-12-03_06H...
7 months ago

Add extra synchronization to test.

This CL adds an additional synchronization point in the test
storage-access-permission.sub.https.window.js. This test installs a
permission-change observer, then initiates a permission change. But the
observer installation is itself asynchronous, and it seems possible the
change the test is trying to observe could occur without the observer
finishing its installation, causing the test to flake. This CL removes
this possibility by forcing the test to wait for the observer
installation to complete before initiating the change the observeer is
intended to observe.

This test has been observed flaking at
https://github.com/web-platform-tests/wpt/runs/27636425026, so it is
hoped this CL will eliminate that flake.

Bug: 40259221
Change-Id: I23e3aa10a3e1e635cc60b17e6ce8445012b6964c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5734161
Commit-Queue: James Maclean wjmaclean@chromium.org
Reviewed-by: Chris Fredrickson cfredric@chromium.org
Cr-Commit-Position: refs/heads/main@{#1332301}

Don't miss a new wpt release

NewReleases is sending notifications on new releases.