Initialize next_update_time_ to allow immediate updating

Initialize next_update_time_ to allow immediate updating

BUG=598412
Committed: https://crrev.com/531801a428d092a4bd50b0e75d5ba03cd1c21612
Cr-Commit-Position: refs/heads/master@{#384112}

[vakh (use Gerrit instead), 2016-03-30 18:24:10]

vakh@chromium.org changed reviewers:
+ nparker@chromium.org

[Nathan Parker, 2016-03-30 20:12:16]

lgtm

Nice find!

https://codereview.chromium.org/1845643002/diff/1/components/safe_browsing_db...
File components/safe_browsing_db/v4_update_protocol_manager.cc (right):

https://codereview.chromium.org/1845643002/diff/1/components/safe_browsing_db...
components/safe_browsing_db/v4_update_protocol_manager.cc:106:
next_update_time_(Time::Now() - base::TimeDelta::FromMilliseconds(1)),
Why not zero?  It's less code.

[vakh (use Gerrit instead), 2016-03-30 21:00:46]

Initialize next_update_time_ with time=0 instead of now-1

[vakh (use Gerrit instead), 2016-03-30 21:04:14]

PTAL. Thanks.

https://codereview.chromium.org/1845643002/diff/1/components/safe_browsing_db...
File components/safe_browsing_db/v4_update_protocol_manager.cc (right):

https://codereview.chromium.org/1845643002/diff/1/components/safe_browsing_db...
components/safe_browsing_db/v4_update_protocol_manager.cc:106:
next_update_time_(Time::Now() - base::TimeDelta::FromMilliseconds(1)),
On 2016/03/30 20:12:16, Nathan Parker wrote:
> Why not zero?  It's less code.

Done.

[Nathan Parker, 2016-03-30 22:03:07]

LGTM still

Meta: My "lgtm + comments" generally mean "address the comments and then feel
free to submit"

[vakh (use Gerrit instead), 2016-03-30 22:03:38]

The CQ bit was checked by vakh@chromium.org

[commit-bot: I haz the power, 2016-03-30 22:04:06]

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

[commit-bot: I haz the power, 2016-03-30 22:10:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-30 22:12:22]

Description was changed from

==========
Initialize next_update_time_ to allow immediate updating

BUG=598412
==========

to

==========
Initialize next_update_time_ to allow immediate updating

BUG=598412

Committed: https://crrev.com/531801a428d092a4bd50b0e75d5ba03cd1c21612
Cr-Commit-Position: refs/heads/master@{#384112}
==========

[commit-bot: I haz the power, 2016-03-30 22:12:23]

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