Bug 227753

Summary: [ Mac Debug ] fast/dom/Window/post-message-large-array-buffer-should-not-crash.html is a flaky failure
Product: WebKit Reporter: ayumi_kojima
Component: New BugsAssignee: ayumi_kojima
Status: NEW    
Severity: Normal CC: keith_miller, tsavell, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=227755
Attachments:
Description Flags
Patch none

ayumi_kojima
Reported 2021-07-07 10:36:16 PDT
fast/dom/Window/post-message-large-array-buffer-should-not-crash.html History: https://results.webkit.org/?suite=layout-tests&test=fast%2Fdom%2FWindow%2Fpost-message-large-array-buffer-should-not-crash.html I was able to reproduce the failure at r279603 and r276913 with run-webkit-tests fast/dom/Window/post-message-large-array-buffer-should-not-crash.html --iterations 50 —debug -f -1 --- /Volumes/Data/Builds/debug-279603/layout-test-results/fast/dom/Window/post-message-large-array-buffer-should-not-crash-expected.txt +++ /Volumes/Data/Builds/debug-279603/layout-test-results/fast/dom/Window/post-message-large-array-buffer-should-not-crash-actual.txt @@ -1,3 +1,4 @@ +CONSOLE MESSAGE: Unhandled Promise Rejection: RangeError: Maximum call stack size exceeded. Check that trying to deserialize an ArrayBuffer when there's not enough memory does not crash (test may only fail flakily)
Attachments
Patch (1.47 KB, patch)
2021-07-07 10:58 PDT, ayumi_kojima 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-07-07 10:37:27 PDT
<rdar://problem/80277602>
ayumi_kojima
Comment 2 2021-07-07 10:58:06 PDT
Created attachment 433048 [details] Patch
Truitt Savell
Comment 3 2021-07-07 11:04:55 PDT
Comment on attachment 433048 [details] Patch Clearing flags on attachment: 433048 Committed r279650 (239463@main): <https://commits.webkit.org/239463@main>
Ryan Haddad
Comment 4 2021-09-09 13:06:13 PDT
*** Bug 227755 has been marked as a duplicate of this bug. ***
Ryan Haddad
Comment 5 2021-09-09 13:06:43 PDT
Comment from Keith in the dupe: We should probably just change the test to "handle" the promise, which can just be an empty callback.
Note You need to log in before you can comment on or make changes to this bug.