Test most of the rest of the LevelDBWrapper public API.

Test most of the rest of the LevelDBWrapper public API.

Adds tests for Delete and DeleteAll, as well as adding verification of
observations. Changes the tests to actually set up a mojo MessagePipe
as well to more realistically test these things.

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

[Marijn Kruisselbrink, 2016-12-21 01:35:23]

The CQ bit was checked by mek@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-21 01:35:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-21 02:21:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-21 02:21:59]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[Marijn Kruisselbrink, 2016-12-21 18:06:35]

The CQ bit was checked by mek@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-21 18:07:20]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Marijn Kruisselbrink, 2016-12-21 18:26:51]

Description was changed from

==========
Improve LevelDBWrapperImpl tests.

Adds tests for Delete and DeleteAll, as well as adding verification of
observations. Changes the tests to actually set up a mojo MessagePipe
as well to more realistically test these things.

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

to

==========
Test most of the rest of the LevelDBWrapper public API.

Adds tests for Delete and DeleteAll, as well as adding verification of
observations. Changes the tests to actually set up a mojo MessagePipe
as well to more realistically test these things.

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

[Marijn Kruisselbrink, 2016-12-21 18:27:31]

mek@chromium.org changed reviewers:
+ jam@chromium.org, michaeln@chromium.org

[commit-bot: I haz the power, 2016-12-21 18:30:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-21 18:30:47]

Dry run: This issue passed the CQ dry run.

[jam, 2016-12-21 21:22:39]

lgtm 👍

[Marijn Kruisselbrink, 2016-12-21 21:27:07]

The CQ bit was checked by mek@chromium.org

[commit-bot: I haz the power, 2016-12-21 21:27:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-21 21:31:30]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482355627613630,
"parent_rev": "ade06bdb35af351f69a60f07154ec8b0a716964e", "commit_rev":
"922fdc7ea9ab5d4bcd9ee8ad05ee31d2ae48143f"}

[commit-bot: I haz the power, 2016-12-21 21:33:02]

Description was changed from

==========
Test most of the rest of the LevelDBWrapper public API.

Adds tests for Delete and DeleteAll, as well as adding verification of
observations. Changes the tests to actually set up a mojo MessagePipe
as well to more realistically test these things.

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

to

==========
Test most of the rest of the LevelDBWrapper public API.

Adds tests for Delete and DeleteAll, as well as adding verification of
observations. Changes the tests to actually set up a mojo MessagePipe
as well to more realistically test these things.

BUG=586194

Review-Url: https://codereview.chromium.org/2588273005
Committed:
https://chromium.googlesource.com/chromium/src/+/922fdc7ea9ab5d4bcd9ee8ad05ee...
==========

[commit-bot: I haz the power, 2016-12-21 21:33:07]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/922fdc7ea9ab5d4bcd9ee8ad05ee...

[commit-bot: I haz the power, 2016-12-21 21:34:31]

Description was changed from

==========
Test most of the rest of the LevelDBWrapper public API.

Adds tests for Delete and DeleteAll, as well as adding verification of
observations. Changes the tests to actually set up a mojo MessagePipe
as well to more realistically test these things.

BUG=586194

Review-Url: https://codereview.chromium.org/2588273005
Committed:
https://chromium.googlesource.com/chromium/src/+/922fdc7ea9ab5d4bcd9ee8ad05ee...
==========

to

==========
Test most of the rest of the LevelDBWrapper public API.

Adds tests for Delete and DeleteAll, as well as adding verification of
observations. Changes the tests to actually set up a mojo MessagePipe
as well to more realistically test these things.

BUG=586194

Committed: https://crrev.com/eea75a43081215ea5e5196fa23d2016ca561f77a
Cr-Commit-Position: refs/heads/master@{#440223}
==========

[commit-bot: I haz the power, 2016-12-21 21:34:32]

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