Chromium Code Reviews Chromium Code Reviews
Issue 2415203002:
Indexed DB: More exception precedence tests (Closed)
| Index: third_party/WebKit/LayoutTests/storage/indexeddb/idbindex-query-exception-order.html |
| diff --git a/third_party/WebKit/LayoutTests/storage/indexeddb/idbindex-query-exception-order.html b/third_party/WebKit/LayoutTests/storage/indexeddb/idbindex-query-exception-order.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..5a91ab1ea66514d897f18d165f86f40d785c36bb |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/storage/indexeddb/idbindex-query-exception-order.html |
| @@ -0,0 +1,73 @@ |
| +<!DOCTYPE html> |
| +<title>IndexedDB: IDBIndex query method Ordering</title> |
| +<meta charset=utf-8> |
| +<link rel="help" href="https://w3c.github.io/IndexedDB/#dom-idbindex-get"> |
| +<link rel="help" href="https://w3c.github.io/IndexedDB/#dom-idbindex-getall"> |
| +<link rel="help" href="https://w3c.github.io/IndexedDB/#dom-idbindex-getallkeys"> |
| +<link rel="help" href="https://w3c.github.io/IndexedDB/#dom-idbindex-count"> |
| +<link rel="help" href="https://w3c.github.io/IndexedDB/#dom-idbindex-opencursor"> |
| +<link rel="help" href="https://w3c.github.io/IndexedDB/#dom-idbindex-openkeycursor"> |
| +<script src="../../resources/testharness.js"></script> |
| +<script src="../../resources/testharnessreport.js"></script> |
| +<script src="resources/testharness-helpers.js"></script> |
| +<script> |
| + |
| +['get', |
| + 'getAll', |
| + 'getAllKeys', |
| + 'count', |
| + 'openCursor', |
| + 'openKeyCursor' |
| +].forEach(method => { |
| + |
| + indexeddb_test( |
| + (t, db) => { |
| + const store = db.createObjectStore('s'); |
| + const store2 = db.createObjectStore('s2'); |
| + const index = store2.createIndex('i', 'keyPath'); |
| + const finish = pin_transaction(store); |
|
pwnall
2016/10/14 07:57:05
I also don't understand why pinning is necessary.
jsbell
2016/10/17 19:42:03
Done.
|
| + |
| + store2.deleteIndex('i'); |
| + |
| + setTimeout(t.step_func(() => { |
| + assert_throws( |
| + 'InvalidStateError', () => { index[method]('key'); }, |
| + '"has been deleted" check (InvalidStateError) should precede ' + |
| + '"not active" check (TransactionInactiveError)'); |
| + |
| + finish(); |
| + t.done(); |
| + }), 0); |
| + }, |
| + (t, db) => {}, |
| + `IDBIndex.${method} exception order: ` + |
| + 'InvalidStateError vs. TransactionInactiveError' |
| + ); |
| + |
| + indexeddb_test( |
| + (t, db) => { |
| + const store = db.createObjectStore('s'); |
| + const index = store.createIndex('i', 'keyPath'); |
| + }, |
| + (t, db) => { |
| + const tx = db.transaction('s'); |
| + const store = tx.objectStore('s'); |
| + const index = store.index('i'); |
| + const finish = pin_transaction(store); |
|
pwnall
2016/10/14 07:57:05
Same comment as above, except this time I commente
jsbell
2016/10/17 19:42:03
Done.
|
| + |
| + setTimeout(t.step_func(() => { |
| + assert_throws( |
| + 'TransactionInactiveError', () => { index[method]({}); }, |
| + '"not active" check (TransactionInactiveError) should precede ' + |
| + 'query check (DataError)'); |
| + |
| + finish(); |
| + t.done(); |
| + }), 0); |
| + }, |
| + `IDBIndex.${method} exception order: ` + |
| + 'TransactionInactiveError vs. DataError' |
| + ); |
| +}); |
| + |
| +</script> |
