IndexedDB: Added IDBObjectStore.getAll() implementation.

content/browser/indexed_db/indexed_db_database.h

Index: content/browser/indexed_db/indexed_db_database.h
diff --git a/content/browser/indexed_db/indexed_db_database.h b/content/browser/indexed_db/indexed_db_database.h
index bcde7190c503bc495e4206f3252a51225424892c..2a02b6f7a42529cda2023810b4f1d1b78393fc6f 100644
--- a/content/browser/indexed_db/indexed_db_database.h
+++ b/content/browser/indexed_db/indexed_db_database.h
@@ -122,6 +122,11 @@ class CONTENT_EXPORT IndexedDBDatabase
            scoped_ptr<IndexedDBKeyRange> key_range,
            bool key_only,
            scoped_refptr<IndexedDBCallbacks> callbacks);
+  void GetAll(int64 transaction_id,
+              int64 object_store_id,

[jsbell, 2015/04/28 00:06:54]

Can we add getAll() for indexes later w/o too much churn across the blink API
boundary?

Similarly, do we want to make GetAllKeys the same method with a different flag,
or a different method?

[cmumford, 2015/04/29 23:17:46]

Good question. I believe it would only be one additional parameter in the
Blink->Chrome direction, much like it is for GetOperation. Personally I think
that GetOperation doesn't benefit by being able to support getting both keys and
values, and my preference would be to split it into two different functions. But
I'm willing to go with what the team considers most readable there (and for
GetAllOperation).

[jsbell, 2015/04/30 00:01:15]

Split is fine, just wanted it thought through.

+              scoped_ptr<IndexedDBKeyRange> key_range,
+              int64 max_count,
+              scoped_refptr<IndexedDBCallbacks> callbacks);
   void Put(int64 transaction_id,
            int64 object_store_id,
            IndexedDBValue* value,
@@ -200,6 +205,11 @@ class CONTENT_EXPORT IndexedDBDatabase
                     indexed_db::CursorType cursor_type,
                     scoped_refptr<IndexedDBCallbacks> callbacks,
                     IndexedDBTransaction* transaction);
+  void GetAllOperation(int64 object_store_id,
+                       scoped_ptr<IndexedDBKeyRange> key_range,
+                       int64 max_count,
+                       scoped_refptr<IndexedDBCallbacks> callbacks,
+                       IndexedDBTransaction* transaction);
   struct PutOperationParams;
   void PutOperation(scoped_ptr<PutOperationParams> params,
                     IndexedDBTransaction* transaction);