Bug 102736 - IndexedDB: migrate backend Index calls to use transaction ids
Summary: IndexedDB: migrate backend Index calls to use transaction ids
Status: RESOLVED DUPLICATE of bug 102741
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alec Flett
URL:
Keywords:
Depends on: 102732 102742 104592
Blocks: 99774
  Show dependency treegraph
 
Reported: 2012-11-19 16:18 PST by Alec Flett
Modified: 2013-01-03 17:08 PST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alec Flett 2012-11-19 16:18:41 PST
Rather than calling through IDBTransactionBackendInterface, backend index calls should just call through IDBDatabase with a transaction id

specifically: (leaving out the details of request/callback hookup/etc)
* IDBIndexBackendInterface::openCursor() becomes 
  IDBDatabaseBackendInterface::openCursor(transaction_id, objectstore_id, index_id, direction, key=false);
* IDBIndexBackendInterface::openKeyCursor() becomes 
  IDBDatabaseBackendInterface::openCursor(transaction_id, objectstore_id, index_id, direction, key=true);
* IDBIndexBackendInterface::count() becomes 
  IDBDatabaseBackendInterface::count(transaction_id, objectstore_id, index_id, keyRange);
* IDBIndexBackendInterface::get() becomes 
  IDBDatabaseBackendInterface::get(transaction_id, objectstore_id, index_id, key=false);
* IDBIndexBackendInterface::getKey() becomes 
  IDBDatabaseBackendInterface::getKey(transaction_id, objectstore_id, index_id, key=false);
Comment 1 Alec Flett 2012-11-19 16:19:22 PST
(erp, that last one should have key=true, and both get() and getKey() take a keyRange too)
Comment 2 Alec Flett 2013-01-03 17:08:30 PST
Decided to just combine these two bugs.

*** This bug has been marked as a duplicate of bug 102741 ***