Upstream an assortment of IndexedDB tests to WPT.

third_party/WebKit/LayoutTests/external/wpt/IndexedDB/support.js

 var databaseName = "database";
 var databaseVersion = 1;
 
 /* Delete created databases
  *
  * Go through each finished test, see if it has an associated database. Close
  * that and delete the database. */
 add_completion_callback(function(tests)
 {
     for (var i in tests)
@@ skipping 44 matching lines @@
         rq_open.addEventListener(evt, function(e) {
             if (current_test !== test) {
                 return;
             }
 
             test.step(function() {
                 if (!rq_open.manually_handled[evt]) {
                     assert_unreached("unexpected open." + evt + " event");
                 }
 
-                if (e.target.result + "" == "[object IDBDatabase]" && !this.db) {
-                    this.db = e.target.result;
+                if (e.target.result + '' == '[object IDBDatabase]' &&
+                    !this.db) {
+                  un this.db = e.target.result;

[jsbell, 2017/01/27 00:31:29]

un -> editing typo?

[pwnall, 2017/01/27 01:23:50]

Done.
Thank you :(
This somehow crawled in after I ran tests locally.

 
-                    this.db.onerror = fail(test, "unexpected db.error");
-                    this.db.onabort = fail(test, "unexpected db.abort");
-                    this.db.onversionchange = fail(test, "unexpected db.versionchange");
+                  this.db.onerror = fail(test, 'unexpected db.error');
+                  this.db.onabort = fail(test, 'unexpected db.abort');
+                  this.db.onversionchange =
+                      fail(test, 'unexpected db.versionchange');
                 }
             })
         })
         rq_open.__defineSetter__("on" + evt, function(h) {
             rq_open.manually_handled[evt] = true;
             if (!h)
                 rq_open.addEventListener(evt, function() {});
             else
                 rq_open.addEventListener(evt, test.step_func(h));
         })
@@ skipping 14 matching lines @@
         }
     });
 
     return rq_open;
 }
 
 function assert_key_equals(actual, expected, description) {
     assert_equals(indexedDB.cmp(actual, expected), 0, description);
 }
 
-function indexeddb_test(upgrade_func, open_func, description) {
-    async_test(function(t) {
-        var dbname = document.location + '-' + t.name;
-        var del = indexedDB.deleteDatabase(dbname);
-        del.onerror = t.unreached_func('deleteDatabase should succeed');
-        var open = indexedDB.open(dbname, 1);
-        open.onerror = t.unreached_func('open should succeed');
-        open.onupgradeneeded = t.step_func(function() {
-            var db = open.result;
-            var tx = open.transaction;
-            upgrade_func(t, db, tx);
-        });
-        open.onsuccess = t.step_func(function() {
-            var db = open.result;
-            if (open_func)
-                open_func(t, db);
-        });
-    }, description);
+function indexeddb_test(upgrade_func, open_func, description, options) {
+  async_test(function(t) {
+    var dbname = document.location + '-' + t.name;
+    var del = indexedDB.deleteDatabase(dbname);
+    del.onerror = t.unreached_func('deleteDatabase should succeed');
+    var open = indexedDB.open(dbname, 1);
+    if (!(options && options.will_abort)) {

[jsbell, 2017/01/27 00:31:29]

Aside: one nice way to handle an options argument in a self-documenting way is
to say near the top:

options = Object.assign({will_abort: false}, options);

This produces a new object with default properties assigned per the first
argument, overwritten with anything from the passed |options|, ignoring
null/undefined.

[pwnall, 2017/01/27 01:23:49]

Done.
I didn't know this pattern. Thank you very much!

+      open.onsuccess = t.unreached_func('open should not succeed');
+    } else {
+      open.onerror = t.unreached_func('open should succeed');
+    }
+    open.onupgradeneeded = t.step_func(function() {
+      var db = open.result;
+      var tx = open.transaction;
+      var old_abort = tx.abort;
+      tx.abort = function() {

[jsbell, 2017/01/27 00:31:29]

we don't want to monkey patch here any more, do we?

[pwnall, 2017/01/27 01:23:50]

Done.
Sorry for the carelessness :(

+        open.onerror = null;
+        old_abort.call(tx);
+      };
+      upgrade_func(t, db, tx);
+    });
+    open.onsuccess = t.step_func(function() {
+      var db = open.result;
+      if (open_func)
+        open_func(t, db);
+    });
+  }, description);
 }