Replace the old URL format for PVer4 requests with the new format.

components/safe_browsing_db/v4_get_hash_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_get_hash_protocol_manager.h"
 
 #include <utility>
 
 #include "base/base64.h"
 #include "base/macros.h"
@@ skipping 299 matching lines @@
     if (gethash_error_count_) {
       RecordGetHashResult(V4OperationResult::BACKOFF_ERROR);
     } else {
       RecordGetHashResult(V4OperationResult::MIN_WAIT_DURATION_ERROR);
     }
     std::vector<SBFullHashResult> full_hashes;
     callback.Run(full_hashes, base::Time());
     return;
   }
 
+  net::HttpRequestHeaders headers;
   std::string req_base64 = GetHashRequest(prefixes, platforms, threat_type);
-  GURL gethash_url = GetHashUrl(req_base64);
+  GURL gethash_url = GetHashUrlAndUpdateHeaders(req_base64, &headers);
 
   net::URLFetcher* fetcher =
       net::URLFetcher::Create(url_fetcher_id_++, gethash_url,
                               net::URLFetcher::GET, this)
           .release();
+  fetcher->SetExtraRequestHeaders(headers.ToString());
   hash_requests_[fetcher] = callback;
 
   fetcher->SetLoadFlags(net::LOAD_DISABLE_CACHE);
   fetcher->SetRequestContext(request_context_getter_.get());
   fetcher->Start();
 }
 
 void V4GetHashProtocolManager::GetFullHashesWithApis(
     const std::vector<SBPrefix>& prefixes,
     FullHashCallback callback) {
@@ skipping 59 matching lines @@
   hash_requests_.erase(it);
 }
 
 void V4GetHashProtocolManager::HandleGetHashError(const Time& now) {
   DCHECK(CalledOnValidThread());
   base::TimeDelta next = V4ProtocolManagerUtil::GetNextBackOffInterval(
       &gethash_error_count_, &gethash_back_off_mult_);
   next_gethash_time_ = now + next;
 }
 
-GURL V4GetHashProtocolManager::GetHashUrl(const std::string& req_base64) const {
-  return V4ProtocolManagerUtil::GetRequestUrl(req_base64, "encodedFullHashes",
-                                              config_);
+GURL V4GetHashProtocolManager::GetHashUrlAndUpdateHeaders(

[Nathan Parker, 2016/06/09 20:22:18]

nit: What is the "update" mean here?  How about GetHashUrlAndHeaders? Same in
v4protocolmanagerutil.

[vakh (use Gerrit instead), 2016/06/09 21:02:15]

Done.

+    const std::string& req_base64,
+    net::HttpRequestHeaders* headers) const {
+  return V4ProtocolManagerUtil::GetRequestUrlAndUpdateHeaders(
+      req_base64, "fullHashes:find", config_, headers);
 }
 
 
 }  // namespace safe_browsing