commit | 4f328a7f306ae64c0af4f2a4cb5da7f3214f6fc7 | [log] [tgz] |
---|---|---|
author | Matt Falkenhagen <falken@chromium.org> | Thu Sep 27 15:02:01 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Thu Sep 27 15:02:01 2018 |
tree | a0934e8510be89a4348940162a366bc8dd15628c | |
parent | d22862c26b6d08ae2979cd352ea114c74c58636b [diff] |
WPT: service worker: Fix potential race in navigation-redirect.https.html. Before this CL, the test makes a frame which posts a message to the test window. But if the test fails, we might not consume the message with `await` and move on the next test. It's possible the next test gets an unexpected message from the previous test. I think a related issue caused the timeout after the Clients variant was added to the test, since the message handler was created after the frame was created and it was possible the message arrived after that. Fix things by explicitly asking the frame for the message when we're ready for it. Bug: 876223, 778497 Change-Id: I82b8685df2977aeadea7475965ecccf69bd33111 Reviewed-on: https://chromium-review.googlesource.com/1249423 Reviewed-by: Ben Kelly <wanderview@chromium.org> Commit-Queue: Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#594723}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .