Initialize next_update_time_ to allow immediate updating

components/safe_browsing_db/v4_update_protocol_manager.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/safe_browsing_db/v4_update_protocol_manager.h"
 
 #include <utility>
 
 #include "base/base64.h"
 #include "base/macros.h"
@@ skipping 83 matching lines @@
 void V4UpdateProtocolManager::ResetUpdateErrors() {
   update_error_count_ = 0;
   update_back_off_mult_ = 1;
 }
 
 V4UpdateProtocolManager::V4UpdateProtocolManager(
     net::URLRequestContextGetter* request_context_getter,
     const V4ProtocolConfig& config)
     : update_error_count_(0),
       update_back_off_mult_(1),
-      next_update_time_(Time::Now()),
+      // The next_update_time_ is just before now so that we can immediately
+      // fetch an update.
+      next_update_time_(Time::Now() - base::TimeDelta::FromMilliseconds(1)),

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

Why not zero?  It's less code.

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

Done.

       config_(config),
       request_context_getter_(request_context_getter),
       url_fetcher_id_(0) {}
 
 V4UpdateProtocolManager::~V4UpdateProtocolManager() {
 }
 
 std::string V4UpdateProtocolManager::GetUpdateRequest(
     const base::hash_set<UpdateListIdentifier>& lists_to_update,
     const base::hash_map<UpdateListIdentifier, std::string>&
@@ skipping 153 matching lines @@
   next_update_time_ = now + next;
 }
 
 GURL V4UpdateProtocolManager::GetUpdateUrl(
     const std::string& req_base64) const {
   return V4ProtocolManagerUtil::GetRequestUrl(req_base64, "encodedUpdates",
                                               config_);
 }
 
 }  // namespace safe_browsing