Bug 232244

Summary: Regression (r284330): [ macOS wk1 Debug ] webaudio/AudioBuffer/huge-buffer.html is a flaky timeout
Product: WebKit Reporter: ayumi_kojima
Component: Web AudioAssignee: Robin Morisset <rmorisset>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, eric.carlson, ews-watchlist, glenn, jer.noble, philipj, rmorisset, sergio, webkit-bot-watchers-bugzilla, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Mac (Intel)   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=229353
Attachments:
Description Flags
Patch none

Description ayumi_kojima 2021-10-25 09:09:43 PDT
webaudio/AudioBuffer/huge-buffer.html

Is flaky timing out on macOS wk1 Debug.

History: https://results.webkit.org/?suite=layout-tests&test=webaudio%2FAudioBuffer%2Fhuge-buffer.html

Result page: https://build.webkit.org/results/Apple-BigSur-Debug-WK1-Tests/r284777%20(4952)/results.html#

Diff:

--- /Volumes/Data/worker/bigsur-debug-tests-wk1/build/layout-test-results/webaudio/AudioBuffer/huge-buffer-expected.txt
+++ /Volumes/Data/worker/bigsur-debug-tests-wk1/build/layout-test-results/webaudio/AudioBuffer/huge-buffer-actual.txt
@@ -1,3 +1,4 @@
+FAIL: Timed out waiting for notifyDone to be called
 
 PASS # AUDIT TASK RUNNER STARTED.
 PASS Executing "huge buffer"
Comment 1 Radar WebKit Bug Importer 2021-10-25 09:10:26 PDT
<rdar://problem/84616427>
Comment 2 ayumi_kojima 2021-10-25 09:13:55 PDT
According to the history, the test was only taking less than 1 second to run before r284330, but it started taking 29+ second after the change.
Comment 3 ayumi_kojima 2021-10-25 09:17:31 PDT
Marked test expectations https://trac.webkit.org/changeset/284787/webkit
Comment 4 Robin Morisset 2021-11-02 14:18:01 PDT
Created attachment 443132 [details]
Patch
Comment 5 Robin Morisset 2021-11-02 18:21:35 PDT
Comment on attachment 443132 [details]
Patch

thanks Yusuke for the review.
Comment 6 EWS 2021-11-02 18:27:34 PDT
Committed r285191 (243819@main): <https://commits.webkit.org/243819@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 443132 [details].