Bug 194225 - [ Mac WK2 ] REGRESSION (r239333) Flaky ASSERTION FAILED: m_uncommittedState.state == State::Committed on http/tests/cookies/same-site/fetch-after-top-level-navigation-from-cross-origin-page.html
Summary: [ Mac WK2 ] REGRESSION (r239333) Flaky ASSERTION FAILED: m_uncommittedState.s...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
Keywords: InRadar
Depends on:
Reported: 2019-02-04 10:22 PST by Shawn Roberts
Modified: 2019-03-21 13:32 PDT (History)
7 users (show)

See Also:

Patch (1.64 KB, patch)
2019-02-19 14:37 PST, Shawn Roberts
no flags Details | Formatted Diff | Diff
Update TestExpectations (1.76 KB, patch)
2019-02-20 13:20 PST, Shawn Roberts
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Shawn Roberts 2019-02-04 10:22:07 PST
The following layout test is flaky on Mac WK2 Debug


Probable cause:

Appears to be flaky timeout or missing failure as of 239333. 

Reproducible with: 

run-webkit-tests --root d239506 http/tests/cookies/same-site/fetch-after-top-level-navigation-from-cross-origin-page.html --iterations 500 -f --debug --exit-after-n-failures=1 --no-retry-failures

Flakiness Dashboard:


Assertion failure: 

ASSERTION FAILED: m_uncommittedState.state == State::Committed
/Volumes/Data/slave/mojave-debug/build/Source/WebKit/UIProcess/PageLoadState.cpp(298) : void WebKit::PageLoadState::didFinishLoad(const Transaction::Token &)
1   0x10fa349e9 WTFCrash
2   0x119be42cb WTFCrashWithInfo(int, char const*, char const*, int)
3   0x11a33ee3d WebKit::PageLoadState::didFinishLoad(WebKit::PageLoadState::Transaction::Token const&)
4   0x11a4f94c2 WebKit::WebPageProxy::didFinishLoadForFrame(unsigned long long, unsigned long long, WebKit::UserData const&)
Comment 2 Radar WebKit Bug Importer 2019-02-05 09:26:29 PST
Comment 3 Chris Dumez 2019-02-05 09:27:47 PST
Did https://bugs.webkit.org/show_bug.cgi?id=193740 not fix this? It was working for me locally.
Comment 4 Alexey Proskuryakov 2019-02-05 09:57:30 PST
It did not, there's a crash with newer r240967: https://build.webkit.org/builders/Apple%20Mojave%20Debug%20WK2%20(Tests)/builds/1461
Comment 5 Chris Dumez 2019-02-05 10:00:02 PST
(In reply to Alexey Proskuryakov from comment #4)
> It did not, there's a crash with newer r240967:
> https://build.webkit.org/builders/Apple%20Mojave%20Debug%20WK2%20(Tests)/
> builds/1461

Ok, thanks. I'll investigate further.
Comment 6 Shawn Roberts 2019-02-19 14:37:50 PST
Created attachment 362431 [details]
Comment 7 WebKit Commit Bot 2019-02-19 14:40:52 PST
Comment on attachment 362431 [details]

Rejecting attachment 362431 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'apply-attachment', '--no-update', '--non-interactive', 362431, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Logging in as commit-queue@webkit.org...
Fetching: https://bugs.webkit.org/attachment.cgi?id=362431&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=194225&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Processing patch 362431 from bug 194225.
Fetching: https://bugs.webkit.org/attachment.cgi?id=362431
Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Parsed 2 diffs from patch file(s).
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/platform/mac-wk2/TestExpectations
Hunk #1 FAILED at 911.
1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/mac-wk2/TestExpectations.rej

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: https://webkit-queues.webkit.org/results/11208643
Comment 8 Shawn Roberts 2019-02-20 13:20:30 PST
Created attachment 362530 [details]
Update TestExpectations
Comment 9 Truitt Savell 2019-02-20 13:25:00 PST
Comment on attachment 362530 [details]
Update TestExpectations

Clearing flags on attachment: 362530

Committed r241831: <https://trac.webkit.org/changeset/241831>
Comment 10 Chris Dumez 2019-03-20 16:33:10 PDT
Committed r243255: <https://trac.webkit.org/changeset/243255>