IndexedDB: Added IDBIndex.getAll() implementation.

content/browser/indexed_db/indexed_db_database.cc

Index: content/browser/indexed_db/indexed_db_database.cc
diff --git a/content/browser/indexed_db/indexed_db_database.cc b/content/browser/indexed_db/indexed_db_database.cc
index 6fbc77f87b99ee7583e67c1ebe95268c2a52d882..8797de4b377569d3b19bbf02aa801a0ef36894fd 100644
--- a/content/browser/indexed_db/indexed_db_database.cc
+++ b/content/browser/indexed_db/indexed_db_database.cc
@@ -535,6 +535,7 @@ void IndexedDBDatabase::Abort(int64 transaction_id,
 
 void IndexedDBDatabase::GetAll(int64 transaction_id,
                                int64 object_store_id,
+                               int64 index_id,
                                scoped_ptr<IndexedDBKeyRange> key_range,
                                int64 max_count,
                                scoped_refptr<IndexedDBCallbacks> callbacks) {
@@ -548,7 +549,7 @@ void IndexedDBDatabase::GetAll(int64 transaction_id,
 
   transaction->ScheduleTask(
       base::Bind(&IndexedDBDatabase::GetAllOperation, this, object_store_id,
-                 Passed(&key_range), max_count, callbacks));
+                 index_id, Passed(&key_range), max_count, callbacks));
 }
 
 void IndexedDBDatabase::Get(int64 transaction_id,
@@ -739,6 +740,7 @@ void IndexedDBDatabase::GetOperation(
 
 void IndexedDBDatabase::GetAllOperation(
     int64 object_store_id,
+    int64 index_id,
     scoped_ptr<IndexedDBKeyRange> key_range,
     int64 max_count,
     scoped_refptr<IndexedDBCallbacks> callbacks,
@@ -756,10 +758,19 @@ void IndexedDBDatabase::GetAllOperation(
 
   leveldb::Status s;
 
-  scoped_ptr<IndexedDBBackingStore::Cursor> cursor =
-      backing_store_->OpenObjectStoreCursor(
-          transaction->BackingStoreTransaction(), id(), object_store_id,
-          *key_range, blink::WebIDBCursorDirectionNext, &s);
+  scoped_ptr<IndexedDBBackingStore::Cursor> cursor;
+
+  if (index_id == IndexedDBIndexMetadata::kInvalidId) {
+    // ObjectStore Retrieval Operation

[jsbell, 2015/05/14 22:22:50]

nit: This doesn't match the spec's "blah blah Retrieval Operation" steps, so
maybe just say "// Object Store" here and "// Index" below.

[cmumford, 2015/05/15 20:55:07]

Done.

+    cursor = backing_store_->OpenObjectStoreCursor(
+        transaction->BackingStoreTransaction(), id(), object_store_id,
+        *key_range, blink::WebIDBCursorDirectionNext, &s);
+  } else {
+    // Index Referenced Value Retrieval Operation
+    cursor = backing_store_->OpenIndexCursor(
+        transaction->BackingStoreTransaction(), id(), object_store_id, index_id,
+        *key_range, blink::WebIDBCursorDirectionNext, &s);
+  }
 
   if (!s.ok()) {
     DLOG(ERROR) << "Unable to open cursor operation: " << s.ToString();