Migrate data from old localstorage format to new format.

Migrate data from old localstorage format to new format.

If when loading data from LevelDB no data is found for a particular origin
this causes data to be (attempted) to be loaded from SQLite instead.

As soon as any data has been successfully written to (disk backed) LevelDB
the SQLite database for that origin is deleted.

Rather than adding even more callbacks to LevelDBWrapperImpl this changes
all existing callbacks and a few new ones to be part of one Delegate
interface.

BUG=586194
Review-Url: https://codereview.chromium.org/2604273002
Cr-Commit-Position: refs/heads/master@{#444152}
Committed: https://chromium.googlesource.com/chromium/src/+/222214f0eedbdc9dc393a7cb40e370d4751d4fd1

[Marijn Kruisselbrink, 2016-12-29 23:20:45]

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

[commit-bot: I haz the power, 2016-12-29 23:21:03]

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-30 00:29:22]

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

[commit-bot: I haz the power, 2016-12-30 00:29:23]

Dry run: This issue passed the CQ dry run.

[Marijn Kruisselbrink, 2017-01-13 19:46:06]

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

[commit-bot: I haz the power, 2017-01-13 19:46:46]

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

[Marijn Kruisselbrink, 2017-01-13 19:54:01]

Description was changed from

==========
WIP: Migrate data from old localstorage format to new format.

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

to

==========
WIP: Migrate data from old localstorage format to new format.

Rather than adding even more callbacks to LevelDBWrapperImpl this changes
all existing callbacks and a few new ones to be part of one Delegate
interface. 

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

[Marijn Kruisselbrink, 2017-01-13 20:12:48]

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

[commit-bot: I haz the power, 2017-01-13 20:13:38]

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

[Marijn Kruisselbrink, 2017-01-13 20:27:21]

Description was changed from

==========
WIP: Migrate data from old localstorage format to new format.

Rather than adding even more callbacks to LevelDBWrapperImpl this changes
all existing callbacks and a few new ones to be part of one Delegate
interface. 

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

to

==========
Migrate data from old localstorage format to new format.

If when loading data from LevelDB no data is found for a particular origin
this causes data to be (attempted) to be loaded from SQLite instead.

As soon as any data has been successfully written to (disk backed) LevelDB
the SQLite database for that origin is deleted.

Rather than adding even more callbacks to LevelDBWrapperImpl this changes
all existing callbacks and a few new ones to be part of one Delegate
interface.

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

[Marijn Kruisselbrink, 2017-01-13 21:02:16]

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

[commit-bot: I haz the power, 2017-01-13 21:41:50]

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

[commit-bot: I haz the power, 2017-01-13 21:41:51]

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, 2017-01-13 21:52:54]

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

[commit-bot: I haz the power, 2017-01-13 21:53:42]

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, 2017-01-13 23:48:26]

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

[commit-bot: I haz the power, 2017-01-13 23:48:27]

Dry run: This issue passed the CQ dry run.

[jam, 2017-01-13 23:51:16]

ltm, great

https://codereview.chromium.org/2604273002/diff/60001/content/browser/dom_sto...
File content/browser/dom_storage/local_storage_context_mojo_unittest.cc (right):

https://codereview.chromium.org/2604273002/diff/60001/content/browser/dom_sto...
content/browser/dom_storage/local_storage_context_mojo_unittest.cc:613: //
Origin should no longer exist in old storage.
nit: Origin1

[jam, 2017-01-17 19:26:09]

On 2017/01/13 23:51:16, jam wrote:
> ltm, great

oops I misspelled that, lgtm

[Marijn Kruisselbrink, 2017-01-17 19:36:26]

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

[commit-bot: I haz the power, 2017-01-17 19:36:52]

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, 2017-01-17 21:20:33]

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

[commit-bot: I haz the power, 2017-01-17 21:20:34]

Dry run: This issue passed the CQ dry run.

[Marijn Kruisselbrink, 2017-01-17 21:24:20]

The CQ bit was checked by mek@chromium.org

[Marijn Kruisselbrink, 2017-01-17 21:24:20]

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/2604273002/#ps80001
(title: "nit")

[commit-bot: I haz the power, 2017-01-17 21:24:46]

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, 2017-01-17 21:30:55]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1484688260340240,
"parent_rev": "beffc4fe07d6a0ca0964db6fbd895da77c36d455", "commit_rev":
"222214f0eedbdc9dc393a7cb40e370d4751d4fd1"}

[commit-bot: I haz the power, 2017-01-17 21:31:30]

Description was changed from

==========
Migrate data from old localstorage format to new format.

If when loading data from LevelDB no data is found for a particular origin
this causes data to be (attempted) to be loaded from SQLite instead.

As soon as any data has been successfully written to (disk backed) LevelDB
the SQLite database for that origin is deleted.

Rather than adding even more callbacks to LevelDBWrapperImpl this changes
all existing callbacks and a few new ones to be part of one Delegate
interface.

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

to

==========
Migrate data from old localstorage format to new format.

If when loading data from LevelDB no data is found for a particular origin
this causes data to be (attempted) to be loaded from SQLite instead.

As soon as any data has been successfully written to (disk backed) LevelDB
the SQLite database for that origin is deleted.

Rather than adding even more callbacks to LevelDBWrapperImpl this changes
all existing callbacks and a few new ones to be part of one Delegate
interface.

BUG=586194

Review-Url: https://codereview.chromium.org/2604273002
Cr-Commit-Position: refs/heads/master@{#444152}
Committed:
https://chromium.googlesource.com/chromium/src/+/222214f0eedbdc9dc393a7cb40e3...
==========

[commit-bot: I haz the power, 2017-01-17 21:31:31]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/222214f0eedbdc9dc393a7cb40e3...