Fix the issue that client id is not saved when upgrading the offline metadata store

Fix the issue that client id is not saved when upgrading the offline metadata store

BUG=601071
Committed: https://crrev.com/5c4aada470737b1708ab65a92e5116634c67550c
Cr-Commit-Position: refs/heads/master@{#386167}

[jianli, 2016-04-07 23:52:32]

Patchset #1 (id:1) has been deleted

[jianli, 2016-04-07 23:52:50]

jianli@chromium.org changed reviewers:
+ bburns@chromium.org, fgorski@chromium.org

[fgorski, 2016-04-08 05:09:58]

lgtm

https://codereview.chromium.org/1869243002/diff/20001/components/offline_page...
File components/offline_pages/offline_page_metadata_store_impl.cc (right):

https://codereview.chromium.org/1869243002/diff/20001/components/offline_page...
components/offline_pages/offline_page_metadata_store_impl.cc:185:
keys_to_remove->push_back(item.client_id.id);
nit: Add a comment that his is removing the old entry keyed with deprecated id.

[jianli, 2016-04-08 19:06:14]

The CQ bit was checked by jianli@chromium.org

[jianli, 2016-04-08 19:06:15]

The patchset sent to the CQ was uploaded after l-g-t-m from fgorski@chromium.org
Link to the patchset: https://codereview.chromium.org/1869243002/#ps40001
(title: "Address feedback")

[commit-bot: I haz the power, 2016-04-08 19:06:39]

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

[commit-bot: I haz the power, 2016-04-08 19:53:39]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-04-08 19:54:37]

Description was changed from

==========
Fix the issue that client id is not saved when upgrading the offline metadata
store

BUG=601071
==========

to

==========
Fix the issue that client id is not saved when upgrading the offline metadata
store

BUG=601071

Committed: https://crrev.com/5c4aada470737b1708ab65a92e5116634c67550c
Cr-Commit-Position: refs/heads/master@{#386167}
==========

[commit-bot: I haz the power, 2016-04-08 19:54:38]

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