storage/indexeddb/delete-in-upgradeneeded-close-in-open-success.html flaky on mac-wk2

[Ryan Haddad]

storage/indexeddb/delete-in-upgradeneeded-close-in-open-success.html flaky on mac-wk2

Most recent failure:
<https://build.webkit.org/results/Apple%20El%20Capitan%20Debug%20WK2%20(Tests)/r197191%20(3257)/results.html>

Flakiness dashboard:
<https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=storage%2Findexeddb%2Fdelete-in-upgradeneeded-close-in-open-success.html>

--- /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/storage/indexeddb/delete-in-upgradeneeded-close-in-open-success-expected.txt
+++ /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/storage/indexeddb/delete-in-upgradeneeded-close-in-open-success-actual.txt
@@ -24,15 +24,15 @@
 PASS event.newVersion is null
 sawVersionChange = true
 
-deleteBlockedCallback():
-PASS sawVersionChange is true
-sawDeleteBlocked = true
-
 openSuccess():
 PASS sawUpgradeNeeded is true
 db = event.target.result
 PASS db.version is 1
 db.close()
+
+deleteBlockedCallback():
+PASS sawVersionChange is true
+sawDeleteBlocked = true
 
 deleteSuccessCallback():
 PASS sawVersionChange is true

[Brady Eidson]

This callback coming out of order like this is okay.

Makes sense that it's WK2-only, since WK1 has a much more well defined order with the thread-hopping versus process hopping.

Easy fix to the test to make it determinate.

[Brady Eidson]

Created attachment 272760 [details]
Patch v1

[WebKit Commit Bot]

Comment on attachment 272760 [details]
Patch v1

Clearing flags on attachment: 272760

Committed r197513: <http://trac.webkit.org/changeset/197513>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.