Bug 206965 – REGRESSION: (253807) [ macOS iOS ] storage/indexeddb/intversion-long-queue-private.html is flaky failing

• Component: New Bugs
• Status: RESOLVED FIXED

[Jacob Uphoff]

storage/indexeddb/intversion-long-queue-private.html

This test is flaky failing 

I was able to reproduce the failure and found that the failure was introduced in r253807

History:

https://results.webkit.org/?suite=layout-tests&suite=layout-tests&test=storage%2Findexeddb%2Fintversion-long-queue-private.html&test=storage%2Fwebsql%2Fstatement-error-callback.html&platform=ios&platform=mac

Diff:

--- /Volumes/Data/slave/catalina-release-tests-wk2/build/layout-test-results/storage/websql/statement-error-callback-expected.txt
+++ /Volumes/Data/slave/catalina-release-tests-wk2/build/layout-test-results/storage/websql/statement-error-callback-actual.txt
@@ -1,15 +1,7 @@
 CONSOLE MESSAGE: line 64: Web SQL is deprecated. Please use IndexedDB instead.
 CONSOLE MESSAGE: line 67: Exception in statement error callback
-This test confirms that a transaction is immediately rolled back if and only if a statement's error callback throws an exception, returns true, or doesn't return any value.
-PASS - the transaction error callback was invoked.
-PASS - the transaction error callback was invoked.
-PASS - the transaction error callback was invoked.
-PASS - the transaction error callback was invoked.
-PASS - the transaction error callback was invoked.
-PASS - the transaction error callback was invoked.
-PASS - the transaction success callback was invoked.
-PASS - the transaction success callback was invoked.
-PASS - the transaction success callback was invoked.
-PASS - the transaction success callback was invoked.
-Test Complete
+#PID UNRESPONSIVE - WebKitTestRunner (pid 92995)
+FAIL: Timed out waiting for notifyDone to be called
 
+#EOF
+#EOF

[Radar WebKit Bug Importer]

<rdar://problem/59005652>

[Jacob Uphoff]

Created attachment 389184 [details]
Update Test Expectations

[Truitt Savell]

Comment on attachment 389184 [details]
Update Test Expectations

Clearing flags on attachment: 389184

Committed r255384: <https://trac.webkit.org/changeset/255384>

[Sihui Liu]

Created attachment 389747 [details]
Patch

[Sihui Liu]

Created attachment 389809 [details]
Patch

[Ryosuke Niwa]

Comment on attachment 389809 [details]
Patch

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

> LayoutTests/ChangeLog:9
> +        Modify a test to make it less flaky. Based on our current implementation, there is no guarantee that Block event on

How about based on spec? Is there supposed to be an ordering guarantee or not?

[Sihui Liu]

(In reply to Ryosuke Niwa from comment #6)
> Comment on attachment 389809 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=389809&action=review
> 
> > LayoutTests/ChangeLog:9
> > +        Modify a test to make it less flaky. Based on our current implementation, there is no guarantee that Block event on
> 
> How about based on spec? Is there supposed to be an ordering guarantee or
> not?

Specs specify how the ordering would be on one open/delete request.

https://w3c.github.io/IndexedDB/#deleting-a-database 6 and 8 specify the ordering 2 for delete request.

https://w3c.github.io/IndexedDB/#factory-interface and https://w3c.github.io/IndexedDB/#opening specify ensure part of the ordering 1 for open request(UpgradeNeeded event on openRequest2 -> Complete event on VersionChange -> transaction on connection2 -> Success event on openRequest2).

It does not guarantee the ordering of all events on different requests.

[Ryosuke Niwa]

Comment on attachment 389809 [details]
Patch

Sounds good to me.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 389809 [details]:

editing/spelling/spellcheck-async-remove-frame.html bug 158401 (authors: [e-mail], [e-mail], and [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 389809 [details]
Patch

Clearing flags on attachment: 389809

Committed r256235: <https://trac.webkit.org/changeset/256235>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Sihui Liu]

*** Bug 207465 has been marked as a duplicate of this bug. ***

[Ryosuke Niwa]

Nice, the test seems to be no longer flaky.