Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(211)

Unified Diff: third_party/WebKit/LayoutTests/storage/indexeddb/resources/deletedatabase-delayed-by-open-and-versionchange.js

Issue 2084053004: IndexedDB: Defer delete calls when there is a running upgrade (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased, made delete lists contain unique_ptrs Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/storage/indexeddb/resources/deletedatabase-delayed-by-open-and-versionchange.js
diff --git a/third_party/WebKit/LayoutTests/storage/indexeddb/resources/deletedatabase-delayed-by-open-and-versionchange.js b/third_party/WebKit/LayoutTests/storage/indexeddb/resources/deletedatabase-delayed-by-open-and-versionchange.js
index 0f005763d11e640479b84078c93dabe62023e471..29bce0b311cd24ff10e7c093375b64eae4e2f806 100644
--- a/third_party/WebKit/LayoutTests/storage/indexeddb/resources/deletedatabase-delayed-by-open-and-versionchange.js
+++ b/third_party/WebKit/LayoutTests/storage/indexeddb/resources/deletedatabase-delayed-by-open-and-versionchange.js
@@ -33,6 +33,7 @@ function prepareDatabase(evt)
request.onblocked = function deleteDatabaseOnBlocked(evt) {
preamble(evt);
shouldBeTrue("sawVersionChange");
+ evalAndLog("h.close()");
};
request.onsuccess = function deleteDatabaseOnSuccess(evt) {
preamble(evt);
@@ -45,9 +46,4 @@ function onOpenSuccess(evt)
{
preamble(evt);
evalAndLog("h = event.target.result");
- evalAndLog("h.close()");
- debug("Closing too late to prevent the in-flight 'blocked' event");
- // Event ordering between 'success' and 'blocked' is not strictly defined
- // in the spec. This documents current Chromium behavior to detect
- // unexpected changes.
}

Powered by Google App Engine
This is Rietveld 408576698