Explicitly set modification_date when loading server entries from db.

Explicitly set modification_date when loading server entries from db.

Fixes an issue where the modification_date on server cards and profiles
is always set to base::Time::Now().

BUG=614803
Committed: https://crrev.com/914445df51cdeae9a867061c32e3b9bbe8784078
Cr-Commit-Position: refs/heads/master@{#396229}

[Justin Donnelly, 2016-05-25 20:24:25]

jdonnelly@chromium.org changed reviewers:
+ mathp@chromium.org

[Mathieu, 2016-05-25 20:28:46]

lgtm with nit

https://codereview.chromium.org/2008253003/diff/20001/components/autofill/cor...
File components/autofill/core/browser/webdata/autofill_table.cc (right):

https://codereview.chromium.org/2008253003/diff/20001/components/autofill/cor...
components/autofill/core/browser/webdata/autofill_table.cc:956:
profile->set_modification_date(Time());
comment? It's easy to miss the subtlety here and below

[Justin Donnelly, 2016-05-25 20:36:51]

https://codereview.chromium.org/2008253003/diff/20001/components/autofill/cor...
File components/autofill/core/browser/webdata/autofill_table.cc (right):

https://codereview.chromium.org/2008253003/diff/20001/components/autofill/cor...
components/autofill/core/browser/webdata/autofill_table.cc:956:
profile->set_modification_date(Time());
On 2016/05/25 20:28:46, Mathieu Perreault wrote:
> comment? It's easy to miss the subtlety here and below

Done.

[Justin Donnelly, 2016-05-26 18:07:23]

The CQ bit was checked by jdonnelly@chromium.org

[Justin Donnelly, 2016-05-26 18:07:23]

The patchset sent to the CQ was uploaded after l-g-t-m from mathp@chromium.org
Link to the patchset: https://codereview.chromium.org/2008253003/#ps40001
(title: "Add comments to implementation file")

[commit-bot: I haz the power, 2016-05-26 18:07:38]

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

[commit-bot: I haz the power, 2016-05-26 18:12:23]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-26 18:13:44]

Description was changed from

==========
Explicitly set modification_date when loading server entries from db.

Fixes an issue where the modification_date on server cards and profiles
is always set to base::Time::Now().

BUG=614803
==========

to

==========
Explicitly set modification_date when loading server entries from db.

Fixes an issue where the modification_date on server cards and profiles
is always set to base::Time::Now().

BUG=614803

Committed: https://crrev.com/914445df51cdeae9a867061c32e3b9bbe8784078
Cr-Commit-Position: refs/heads/master@{#396229}
==========

[commit-bot: I haz the power, 2016-05-26 18:13:45]

Patchset 3 (id:??) landed as
https://crrev.com/914445df51cdeae9a867061c32e3b9bbe8784078
Cr-Commit-Position: refs/heads/master@{#396229}