WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
206949
[ Mac ] fast/dom/Window/post-message-crash.html is flaky failing
https://bugs.webkit.org/show_bug.cgi?id=206949
Summary
[ Mac ] fast/dom/Window/post-message-crash.html is flaky failing
Jason Lawrence
Reported
2020-01-29 10:48:58 PST
fast/dom/Window/post-message-crash.html Description: This test is flaky failing on Mac. I see a potential regression point of 252824. History:
https://results.webkit.org/?suite=layout-tests&test=fast%2Fdom%2FWindow%2Fpost-message-crash.html
Diff: --- /Volumes/Data/slave/catalina-leaks/build/layout-test-results/fast/dom/Window/post-message-crash-expected.txt +++ /Volumes/Data/slave/catalina-leaks/build/layout-test-results/fast/dom/Window/post-message-crash-actual.txt @@ -1,3 +1,3 @@ This ensures that postMessage called in a callback does not crash. -PASS +FAIL
Attachments
Update Test Expectations
(1.34 KB, patch)
2020-01-29 10:54 PST
,
Jason Lawrence
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2020-01-29 10:49:24 PST
<
rdar://problem/58997453
>
Jason Lawrence
Comment 2
2020-01-29 10:54:48 PST
Created
attachment 389157
[details]
Update Test Expectations
Truitt Savell
Comment 3
2020-01-29 11:14:34 PST
Comment on
attachment 389157
[details]
Update Test Expectations Clearing flags on attachment: 389157 Committed
r255370
: <
https://trac.webkit.org/changeset/255370
>
Alexey Proskuryakov
Comment 4
2020-01-29 23:09:09 PST
This test was already marked as flaky on iOS. Clearly a regression on Mac, at least in frequency.
Chris Dumez
Comment 5
2020-01-30 08:50:58 PST
The test relies on a 50ms timer :( setTimeout(function() { if (window.testRunner) testRunner.notifyDone(); }, 50);
Chris Dumez
Comment 6
2020-01-30 08:56:17 PST
Committed
r255427
: <
https://trac.webkit.org/changeset/255427
>
Alexey Proskuryakov
Comment 7
2020-01-30 09:00:24 PST
This would have helped help if the test lacked output, but that's not the failure seen here.
Alexey Proskuryakov
Comment 8
2020-01-30 09:04:00 PST
I was wrong, the test replaces FAIL with PASS.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug