IndexedDB: Added IDBIndex.getAll() implementation.

IndexedDB: Added IDBIndex.getAll() implementation.

This is the experimental IDBIndex.getAll implementation for
retrieving multiple values in a single request as proposed in
https://www.w3.org/2008/webapps/wiki/IndexedDatabaseFeatures

Mozilla's documentation for IDBIndex.getAll is at:
https://developer.mozilla.org/en-US/docs/Web/API/IDBIndex/getAll

In order to use run Chrome with: --enable-experimental-web-platform-features

BUG=457450
Committed: https://crrev.com/75696ff695438f938582ffee66c705015d4249bb
Cr-Commit-Position: refs/heads/master@{#330631}

[cmumford, 2015-05-14 20:49:49]

cmumford@chromium.org changed reviewers:
+ jsbell@chromium.org

[cmumford, 2015-05-14 20:52:23]

Oh - just remembered. I also want to add a corruption test for
IDBIndex.getAll(). Another patch set to follow shortly.

[cmumford, 2015-05-14 21:18:51]

A corruption test depends on changes in
https://codereview.chromium.org/1116253003, so I will followup this CL with a
separate corruption test CL.

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

lgtm, will need tsepez@ to approve the messaging change.

https://codereview.chromium.org/1147433002/diff/1/content/browser/indexed_db/...
File content/browser/indexed_db/indexed_db_database.cc (right):

https://codereview.chromium.org/1147433002/diff/1/content/browser/indexed_db/...
content/browser/indexed_db/indexed_db_database.cc:764: // ObjectStore Retrieval
Operation
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]

cmumford@chromium.org changed reviewers:
+ tsepez@chromium.org

[cmumford, 2015-05-15 20:55:08]

+tsepez to review content/common/indexed_db/indexed_db_messages.h

https://codereview.chromium.org/1147433002/diff/1/content/browser/indexed_db/...
File content/browser/indexed_db/indexed_db_database.cc (right):

https://codereview.chromium.org/1147433002/diff/1/content/browser/indexed_db/...
content/browser/indexed_db/indexed_db_database.cc:764: // ObjectStore Retrieval
Operation
On 2015/05/14 22:22:50, jsbell wrote:
> nit: This doesn't match the spec's "blah blah Retrieval Operation" steps, so
> maybe just say "// Object Store" here and "// Index" below.

Done.

[Tom Sepez, 2015-05-15 21:00:07]

Messages LGTM

[cmumford, 2015-05-19 21:27:53]

The CQ bit was checked by cmumford@chromium.org

[cmumford, 2015-05-19 21:27:54]

The patchset sent to the CQ was uploaded after l-g-t-m from jsbell@chromium.org
Link to the patchset: https://codereview.chromium.org/1147433002/#ps20001
(title: "Tweaked getAll comments in indexed_db_database.cc")

[commit-bot: I haz the power, 2015-05-19 21:28:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1147433002/20001

[commit-bot: I haz the power, 2015-05-19 22:22:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-19 22:23:07]

Patchset 2 (id:??) landed as
https://crrev.com/75696ff695438f938582ffee66c705015d4249bb
Cr-Commit-Position: refs/heads/master@{#330631}