Summary: | [High Sierra Release WK2] LayoutTest http/tests/loading/resourceLoadStatistics/partitioned-cookies-with-and-without-user-interaction.html is a flaky failure | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryan Haddad <ryanhaddad> | ||||
Component: | Tools / Tests | Assignee: | Ryan Haddad <ryanhaddad> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ap, commit-queue, jlewis3, lforschler, webkit-bug-importer, wilander | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Ryan Haddad
2017-09-29 13:24:09 PDT
I suspect that this is a mistake in the test, which races parsing and navigation. Moving runTest() from inline script to load event handler may fix the problem. Ryan or Matt, would you be willing to try this out? (In reply to Alexey Proskuryakov from comment #1) > I suspect that this is a mistake in the test, which races parsing and > navigation. Moving runTest() from inline script to load event handler may > fix the problem. > > Ryan or Matt, would you be willing to try this out? I'll look at this. I can reproduce the failure with: run-webkit-tests --release http/tests/loading/resourceLoadStatistics/ --no-retry-failure -fg I cannot reproduce the failure after moving runTest() as suggested. Created attachment 322745 [details]
Patch
Comment on attachment 322745 [details] Patch Clearing flags on attachment: 322745 Committed r223008: <http://trac.webkit.org/changeset/223008> All reviewed patches have been landed. Closing bug. |