1 files changed, 293 insertions, 0 deletions
diff --git a/testing/web-platform/tests/IndexedDB/transaction-abort-multiple-metadata-revert.html b/testing/web-platform/tests/IndexedDB/transaction-abort-multiple-metadata-revert.html
new file mode 100644
index 0000000000..18abd0588c
--- /dev/null
+++ b/testing/web-platform/tests/IndexedDB/transaction-abort-multiple-metadata-revert.html
@@ -0,0 +1,293 @@
+<!doctype html>
+<meta charset="utf8">
+<title>IndexedDB: aborting transactions reverts multiple operations on the same metadata</title>
+<link rel="help" href="https://w3c.github.io/IndexedDB/#abort-transaction">
+<link rel="author" href="pwnall@chromium.org" title="Victor Costan">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="resources/support-promises.js"></script>
+<script>
+'use strict';
+
+promise_test(testCase => {
+    let store = null, index = null;
+    let migrationTransaction = null, migrationDatabase = null;
+    return createDatabase(testCase, (database, transaction) => {
+        createBooksStore(testCase, database);
+    }).then(database => {
+        database.close();
+    }).then(() => migrateDatabase(testCase, 2, (database, transaction) => {
+        store = createNotBooksStore(testCase, database);
+        migrationDatabase = database;
+        migrationTransaction = transaction;
+        assert_array_equals(
+            database.objectStoreNames, ['books', 'not_books'],
+            'IDBDatabase.objectStoreNames should include a newly created ' +
+            'store before the transaction is aborted');
+        assert_array_equals(
+            transaction.objectStoreNames, ['books', 'not_books'],
+            'IDBTransaction.objectStoreNames should include a newly created ' +
+            'store before the transaction is aborted');
+
+        index = store.index('not_by_author');
+        store.deleteIndex('not_by_author');
+        assert_throws_dom(
+            'InvalidStateError', () => index.get('query'),
+            'IDBIndex.get should throw InvalidStateError, indicating that ' +
+            'the index is marked for deletion, immediately after ' +
+            'IDBObjectStore.deleteIndex() returns');
+        assert_array_equals(
+            store.indexNames, ['not_by_title'],
+            'IDBObjectStore.indexNames should not include the deleted index ' +
+            'immediately after IDBObjectStore.deleteIndex() returns');
+
+        transaction.abort();
+        assert_throws_dom(
+            'InvalidStateError', () => store.get('query'),
+            'IDBObjectStore.get should throw InvalidStateError, indicating ' +
+            'that the store is marked for deletion, immediately after ' +
+            'IDBTransaction.abort() returns');
+        assert_throws_dom(
+            'InvalidStateError', () => index.get('query'),
+            'IDBIndex.get should throw InvalidStateError, indicating that ' +
+            'the index is still marked for deletion, immediately after ' +
+            'IDBTransaction.abort() returns');
+        assert_array_equals(
+            transaction.objectStoreNames, ['books'],
+            'IDBTransaction.objectStoreNames should stop including the newly ' +
+            'created store immediately after IDBTransaction.abort() returns');
+        assert_array_equals(
+            database.objectStoreNames, ['books'],
+            'IDBDatabase.objectStoreNames should stop including the newly ' +
+            'created store immediately after IDBTransaction.abort() returns');
+        assert_array_equals(
+            store.indexNames, [],
+            'IDBObjectStore.indexNames for the newly created store should be ' +
+            'empty immediately after IDBTransaction.abort() returns');
+    })).then(() => {
+        assert_throws_dom(
+            'InvalidStateError', () => store.get('query'),
+            'IDBObjectStore.get should throw InvalidStateError, indicating ' +
+            'that the store is marked for deletion, after the transaction is ' +
+            'aborted');
+        assert_throws_dom(
+            'InvalidStateError', () => index.get('query'),
+            'IDBIndex.get should throw InvalidStateError, indicating that ' +
+            'the index is still marked for deletion, after the transaction ' +
+            'is aborted');
+        assert_array_equals(
+            migrationDatabase.objectStoreNames, ['books'],
+            'IDBDatabase.objectStoreNames should stop including the newly ' +
+            'created store after the transaction is aborted');
+        assert_array_equals(
+            migrationTransaction.objectStoreNames, ['books'],
+            'IDBTransaction.objectStoreNames should stop including the newly ' +
+            'created store after the transaction is aborted');
+        assert_array_equals(
+            store.indexNames, [],
+            'IDBObjectStore.indexNames for the newly created store should be ' +
+            'empty after the transaction is aborted');
+    });
+}, 'Deleted indexes in newly created stores are still marked as deleted ' +
+    'after the transaction aborts');
+
+promise_test(testCase => {
+    let store = null, index = null;
+    let migrationTransaction = null, migrationDatabase = null;
+    return createDatabase(testCase, (database, transaction) => {
+        createBooksStore(testCase, database);
+        createNotBooksStore(testCase, database);
+    }).then(database => {
+        database.close();
+    }).then(() => migrateDatabase(testCase, 2, (database, transaction) => {
+        migrationDatabase = database;
+        migrationTransaction = transaction;
+        store = transaction.objectStore('not_books');
+        index = store.index('not_by_author');
+        store.deleteIndex('not_by_author');
+        assert_throws_dom(
+            'InvalidStateError', () => index.get('query'),
+            'IDBIndex.get should throw InvalidStateError, indicating that ' +
+            'the index is marked for deletion, immediately after ' +
+            'IDBObjectStore.deleteIndex() returns');
+        assert_array_equals(
+            store.indexNames, ['not_by_title'],
+            'IDBObjectStore.indexNames should not include the deleted index ' +
+            'immediately after IDBObjectStore.deleteIndex() returns');
+
+        database.deleteObjectStore('not_books');
+        assert_throws_dom(
+            'InvalidStateError', () => store.get('query'),
+            'IDBObjectStore.get should throw InvalidStateError, indicating ' +
+            'that the store is marked for deletion, immediately after ' +
+            'IDBDatabase.deleteObjectStore() returns');
+        assert_throws_dom(
+            'InvalidStateError', () => index.get('query'),
+            'IDBIndex.get should throw InvalidStateError, indicating that ' +
+            'the index is still marked for deletion, immediately after ' +
+            'IDBObjectStore.deleteIndex() returns');
+        assert_array_equals(
+            transaction.objectStoreNames, ['books'],
+            'IDBTransaction.objectStoreNames should stop including the ' +
+            'deleted store immediately after IDBDatabase.deleteObjectStore() ' +
+            'returns');
+        assert_array_equals(
+            database.objectStoreNames, ['books'],
+            'IDBDatabase.objectStoreNames should stop including the newly ' +
+            'created store immediately after IDBDatabase.deleteObjectStore() ' +
+            'returns');
+        assert_array_equals(
+            store.indexNames, [],
+            'IDBObjectStore.indexNames for the deleted store should be empty ' +
+            'immediately after IDBDatabase.deleteObjectStore() returns');
+
+        transaction.abort();
+        assert_throws_dom(
+            'TransactionInactiveError', () => store.get('query'),
+            'IDBObjectStore.get should throw TransactionInactiveError, ' +
+            'indicating that the store is no longer marked for deletion, ' +
+            'immediately after IDBTransaction.abort() returns');
+        assert_throws_dom(
+            'TransactionInactiveError', () => index.get('query'),
+            'IDBIndex.get should throw TransactionInactiveError, indicating ' +
+            'that the index is no longer marked for deletion, immediately ' +
+            'after IDBObjectStore.deleteIndex() returns');
+        assert_array_equals(
+            database.objectStoreNames, ['books', 'not_books'],
+            'IDBDatabase.objectStoreNames should include the deleted store ' +
+            'store immediately after IDBTransaction.abort() returns');
+        assert_array_equals(
+            transaction.objectStoreNames, ['books', 'not_books'],
+            'IDBTransaction.objectStoreNames should include the deleted ' +
+            'store immediately after IDBTransaction.abort() returns');
+        assert_array_equals(
+            store.indexNames, ['not_by_author', 'not_by_title'],
+            'IDBObjectStore.indexNames for the deleted store should not be ' +
+            'empty any more immediately after IDBTransaction.abort() returns');
+    })).then(() => {
+        assert_throws_dom(
+            'TransactionInactiveError', () => store.get('query'),
+            'IDBObjectStore.get should throw TransactionInactiveError, ' +
+            'indicating that the store is no longer marked for deletion, ' +
+            'after the transaction is aborted');
+        assert_throws_dom(
+            'TransactionInactiveError', () => index.get('query'),
+            'IDBIndex.get should throw TransactionInactiveError, indicating ' +
+            'that the index is no longer marked for deletion, after the ' +
+            'transaction is aborted');
+        assert_array_equals(
+            migrationDatabase.objectStoreNames, ['books', 'not_books'],
+            'IDBDatabase.objectStoreNames should include the previously ' +
+            'deleted store after the transaction is aborted');
+        assert_array_equals(
+            migrationTransaction.objectStoreNames, ['books', 'not_books'],
+            'IDBTransaction.objectStoreNames should include the previously ' +
+            'deleted store after the transaction is aborted');
+        assert_array_equals(
+            store.indexNames, ['not_by_author', 'not_by_title'],
+            'IDBObjectStore.indexNames for the deleted store should not be ' +
+            'empty after the transaction is aborted');
+    });
+}, 'Deleted indexes in deleted stores are still marked as not-deleted after ' +
+    'the transaction aborts');
+
+promise_test(testCase => {
+    let store = null, index = null;
+    let migrationTransaction = null, migrationDatabase = null;
+    return createDatabase(testCase, (database, transaction) => {
+        createBooksStore(testCase, database);
+    }).then(database => {
+        database.close();
+    }).then(() => migrateDatabase(testCase, 2, (database, transaction) => {
+        store = createNotBooksStore(testCase, database);
+        migrationDatabase = database;
+        migrationTransaction = transaction;
+        index = store.index('not_by_author');
+        store.deleteIndex('not_by_author');
+        assert_throws_dom(
+            'InvalidStateError', () => index.get('query'),
+            'IDBIndex.get should throw InvalidStateError, indicating that ' +
+            'the index is marked for deletion, immediately after ' +
+            'IDBObjectStore.deleteIndex() returns');
+        assert_array_equals(
+            store.indexNames, ['not_by_title'],
+            'IDBObjectStore.indexNames should not include the deleted index ' +
+            'immediately after IDBObjectStore.deleteIndex() returns');
+
+        database.deleteObjectStore('not_books');
+        assert_throws_dom(
+            'InvalidStateError', () => store.get('query'),
+            'IDBObjectStore.get should throw InvalidStateError, indicating ' +
+            'that the store is marked for deletion, immediately after ' +
+            'IDBDatabase.deleteObjectStore() returns');
+        assert_throws_dom(
+            'InvalidStateError', () => index.get('query'),
+            'IDBIndex.get should throw InvalidStateError, indicating that ' +
+            'the index is still marked for deletion, immediately after ' +
+            'IDBDatabase.deleteObjectStore() returns');
+        assert_array_equals(
+            transaction.objectStoreNames, ['books'],
+            'IDBTransaction.objectStoreNames should stop including the ' +
+            'deleted store immediately after IDBDatabase.deleteObjectStore() ' +
+            'returns');
+        assert_array_equals(
+            database.objectStoreNames, ['books'],
+            'IDBDatabase.objectStoreNames should stop including the newly ' +
+            'created store immediately after IDBDatabase.deleteObjectStore() ' +
+            'returns');
+        assert_array_equals(
+            store.indexNames, [],
+            'IDBObjectStore.indexNames should be empty immediately after ' +
+            'IDBDatabase.deleteObjectStore() returns');
+
+        transaction.abort();
+        assert_throws_dom(
+            'InvalidStateError', () => store.get('query'),
+            'IDBObjectStore.get should throw InvalidStateError, indicating ' +
+            'that the store is still marked for deletion, immediately after ' +
+            'IDBTransaction.abort() returns');
+        assert_throws_dom(
+            'InvalidStateError', () => index.get('query'),
+            'IDBIndex.get should throw InvalidStateError, indicating that ' +
+            'the index is still marked for deletion, immediately after ' +
+            'IDBTransaction.abort() returns');
+        assert_array_equals(
+            transaction.objectStoreNames, ['books'],
+            'IDBTransaction.objectStoreNames should not include the newly ' +
+            'created store immediately after IDBTransaction.abort() returns');
+        assert_array_equals(
+            database.objectStoreNames, ['books'],
+            'IDBDatabase.objectStoreNames should not include the newly ' +
+            'created store immediately after IDBTransaction.abort() returns');
+        assert_array_equals(
+            store.indexNames, [],
+            'IDBObjectStore.indexNames should be empty immediately after ' +
+            'IDBTransaction.abort() returns');
+    })).then(() => {
+        assert_throws_dom(
+            'InvalidStateError', () => store.get('query'),
+            'IDBObjectStore.get should throw InvalidStateError, indicating ' +
+            'that the store is still marked for deletion, after the ' +
+            'transaction is aborted');
+        assert_throws_dom(
+            'InvalidStateError', () => index.get('query'),
+            'IDBIndex.get should throw InvalidStateError, indicating that ' +
+            'the index is still marked for deletion, after the transaction ' +
+            'is aborted');
+        assert_array_equals(
+            migrationDatabase.objectStoreNames, ['books'],
+            'IDBDatabase.objectStoreNames should not include the newly ' +
+            'created store after the transaction is aborted');
+        assert_array_equals(
+            migrationTransaction.objectStoreNames, ['books'],
+            'IDBTransaction.objectStoreNames should not include the newly ' +
+            'created store after the transaction is aborted');
+        assert_array_equals(
+            store.indexNames, [],
+            'IDBObjectStore.indexNames should be empty after the transaction ' +
+            'is aborted');
+    });
+}, 'Deleted indexes in created+deleted stores are still marked as deleted ' +
+    'after their transaction aborts');
+
+</script>