IndexedDB: Align abort behavior on uncaught exception with Gecko

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 125 matching lines @@
 function expect(t, expected) {
   var results = [];
   return result => {
     results.push(result);
     if (results.length === expected.length) {
       assert_array_equals(results, expected);
       t.done();
     }
   };
 }
+
+// Checks to see if the passed transaction is active (by
+// making
+// requests against the named store). Returns a function to
+// to let the transaction finish.

[pwnall, 2017/03/04 00:27:33]

double "to", and I don't think this comment is accurate

[jsbell, 2017/03/06 18:25:02]

Thanks. And whoah, not sure what happened there.

+function is_transaction_active(tx, store_name) {
+  try {
+    const request = tx.objectStore(store_name).get(0);
+    request.onerror = e => {
+      e.preventDefault();
+      e.stopPropagation();
+    };
+    return true;
+  } catch (ex) {
+    assert_equals(ex.name, 'TransactionInactiveError',
+                  'Active check should either not throw, or throw ' +

[pwnall, 2017/03/04 00:27:33]

"not throw anything" would make the structure parallel with "should throw ...",
and is a bit easier to parse for me.

[jsbell, 2017/03/06 18:25:02]

Done.

+                  'TransactionInactiveError');
+    return false;
+  }
+}