mojo leveldb: Add an iteration interface.

mojo leveldb: Add an iteration interface.

This adds iteration support to the mojo:leveldb interface, and builds a
compatibility iterator that synchronously blocks for data so that we
don't have to rewrite everything to the asynchronous style all at once.

BUG=586194
Committed: https://crrev.com/6060a04b7320166f8fad82f39df7969044ab253b
Cr-Commit-Position: refs/heads/master@{#385854}

[Elliot Glaysher, 2016-04-06 20:35:29]

Description was changed from

==========
mojo leveldb: Add an iteration interface.

This adds iteration support to the mojo:leveldb interface, and builds a
compatibility iterator that synchronously blocks for data so that we
don't have to rewrite everything to the asynchronous style all at once.

BUG=
==========

to

==========
mojo leveldb: Add an iteration interface.

This adds iteration support to the mojo:leveldb interface, and builds a
compatibility iterator that synchronously blocks for data so that we
don't have to rewrite everything to the asynchronous style all at once.

BUG=586194
==========

[Elliot Glaysher, 2016-04-06 20:35:29]

erg@chromium.org changed reviewers:
+ jam@chromium.org

[jam, 2016-04-07 16:06:01]

lgtm

https://codereview.chromium.org/1861903002/diff/60001/components/leveldb/publ...
File components/leveldb/public/cpp/util.h (right):

https://codereview.chromium.org/1861903002/diff/60001/components/leveldb/publ...
components/leveldb/public/cpp/util.h:17: leveldb::Status
DatabaseErrorToStatus(DatabaseError e,
nit: please add a comment for all these public methods

[Elliot Glaysher, 2016-04-07 17:47:22]

The CQ bit was checked by erg@chromium.org

[Elliot Glaysher, 2016-04-07 17:47:22]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/1861903002/#ps80001
(title: "Rebase + comments")

[commit-bot: I haz the power, 2016-04-07 17:48:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1861903002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1861903002/80001

[commit-bot: I haz the power, 2016-04-07 21:15:09]

Description was changed from

==========
mojo leveldb: Add an iteration interface.

This adds iteration support to the mojo:leveldb interface, and builds a
compatibility iterator that synchronously blocks for data so that we
don't have to rewrite everything to the asynchronous style all at once.

BUG=586194
==========

to

==========
mojo leveldb: Add an iteration interface.

This adds iteration support to the mojo:leveldb interface, and builds a
compatibility iterator that synchronously blocks for data so that we
don't have to rewrite everything to the asynchronous style all at once.

BUG=586194
==========

[commit-bot: I haz the power, 2016-04-07 21:15:10]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-04-07 21:29:30]

Description was changed from

==========
mojo leveldb: Add an iteration interface.

This adds iteration support to the mojo:leveldb interface, and builds a
compatibility iterator that synchronously blocks for data so that we
don't have to rewrite everything to the asynchronous style all at once.

BUG=586194
==========

to

==========
mojo leveldb: Add an iteration interface.

This adds iteration support to the mojo:leveldb interface, and builds a
compatibility iterator that synchronously blocks for data so that we
don't have to rewrite everything to the asynchronous style all at once.

BUG=586194

Committed: https://crrev.com/6060a04b7320166f8fad82f39df7969044ab253b
Cr-Commit-Position: refs/heads/master@{#385854}
==========

[commit-bot: I haz the power, 2016-04-07 21:29:31]

Patchset 5 (id:??) landed as
https://crrev.com/6060a04b7320166f8fad82f39df7969044ab253b
Cr-Commit-Position: refs/heads/master@{#385854}