Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: fix race in test Frame.waitForFunction should work when context is destroyed #9368

Merged
merged 1 commit into from
Dec 6, 2022

Conversation

whimboo
Copy link
Collaborator

@whimboo whimboo commented Dec 6, 2022

Fixes the race condition which causes intermittent failures in Firefox because we haven't implemented bootstrap scripts to run on document creation.

@whimboo
Copy link
Collaborator Author

whimboo commented Dec 6, 2022

@OrKoN would you mind to review? Thanks!

@whimboo whimboo requested a review from OrKoN December 6, 2022 16:34
@OrKoN OrKoN enabled auto-merge (squash) December 6, 2022 16:47
@OrKoN OrKoN merged commit be7626b into puppeteer:main Dec 6, 2022
@whimboo whimboo deleted the wait_task_disposal branch December 6, 2022 17:01
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Dec 7, 2022
…en resolved right before execution context disposal" to fail r=webdriver-reviewers,jdescottes

The test is currently broken and got fixed upstream for
a later Puppeteer release:

puppeteer/puppeteer#9368

Until the fix got downstreamed this test will be flaky
and can pass and fail.

Differential Revision: https://phabricator.services.mozilla.com/D163984
jamienicol pushed a commit to jamienicol/gecko that referenced this pull request Dec 8, 2022
…en resolved right before execution context disposal" to fail r=webdriver-reviewers,jdescottes

The test is currently broken and got fixed upstream for
a later Puppeteer release:

puppeteer/puppeteer#9368

Until the fix got downstreamed this test will be flaky
and can pass and fail.

Differential Revision: https://phabricator.services.mozilla.com/D163984
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants