[ macOS wk2 ARM 64 ] storage/indexeddb/IDBObject-leak.html is a flakey text failure

• Component: New Bugs
• Status: RESOLVED FIXED

[Robert Jenner]

storage/indexeddb/IDBObject-leak.html

is a flakey text failure in macOS BigSur Release wk2 in Apple Silicon only. 

HISTORY URL:
https://results.webkit.org/?suite=layout-tests&test=storage%2Findexeddb%2FIDBObject-leak.html

TEXT DIFF:
--- /Volumes/Data/worker/bigsur-release-applesilicon-tests-wk2/build/layout-test-results/storage/indexeddb/IDBObject-leak-expected.txt
+++ /Volumes/Data/worker/bigsur-release-applesilicon-tests-wk2/build/layout-test-results/storage/indexeddb/IDBObject-leak-actual.txt
@@ -3,8 +3,9 @@
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
-PASS internals.numberOfIDBTransactions() is 0
+FAIL internals.numberOfIDBTransactions() should be 0. Was 1.
 PASS successfullyParsed is true
+Some tests failed.
 
 TEST COMPLETE

[Radar WebKit Bug Importer]

<rdar://problem/75580609>

[Robert Jenner]

Updated test expectations to Pass Failure here:
https://trac.webkit.org/changeset/274651/webkit

[Robert Jenner]

The test only appears to be flakey failing on Apple Silicon as of lately. As it is only occurring on Apple Silicon Macs, I cannot reproduce the failure as I do not have access to one.

[Sihui Liu]

Created attachment 425341 [details]
Patch

[Sihui Liu]

Created attachment 434602 [details]
Patch

[youenn fablet]

Comment on attachment 434602 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=434602&action=review

> LayoutTests/ChangeLog:12
> +        IDBTransaction. 

Do we really need more than one transaction? If we call GC multiple times asynchronously, I would hope one single transaction would be ok.

Also, should we make the test stress compatible?

[Sihui Liu]

(In reply to youenn fablet from comment #6)
> Comment on attachment 434602 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=434602&action=review
> 
> > LayoutTests/ChangeLog:12
> > +        IDBTransaction. 
> 
> Do we really need more than one transaction? If we call GC multiple times
> asynchronously, I would hope one single transaction would be ok.
> 
> Also, should we make the test stress compatible?

I will investigate why the stress test fails.

[Sihui Liu]

Created attachment 434652 [details]
Patch

[Sihui Liu]

Created attachment 434665 [details]
Patch

[Sihui Liu]

Created attachment 434670 [details]
Patch

[EWS]

Committed r280510 (240142@main): <https://commits.webkit.org/240142@main>

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