Set client_id and client_version in requests sent to SafeBrowsing

components/safe_browsing_db/v4_get_hash_protocol_manager_unittest.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 <memory>
 #include <vector>
 
 #include "base/base64.h"
@@ skipping 264 matching lines @@
 
   pm->UpdateCache(prefixes_requested, fhis, negative_cache_expire);
 
   // Verify the state of the cache.
   const FullHashCache* cache = pm->full_hash_cache_for_tests();
   // Check the cache.
   EXPECT_EQ(0u, cache->size());
 }
 
 TEST_F(V4GetHashProtocolManagerTest, TestGetHashRequest) {
-  HashPrefix one = "hashone";
-  HashPrefix two = "hashtwo";
-  std::vector<HashPrefix> prefixes_to_request = {one, two};
-
   FindFullHashesRequest req;
   ThreatInfo* info = req.mutable_threat_info();
-  info->add_threat_types(MALWARE_THREAT);
-  info->add_threat_types(API_ABUSE);
-
   info->add_platform_types(GetCurrentPlatformType());
   info->add_platform_types(CHROME_PLATFORM);
 
   info->add_threat_entry_types(URL);
 
+  info->add_threat_types(MALWARE_THREAT);
+  info->add_threat_types(API_ABUSE);
+
+  HashPrefix one = "hashone";
+  HashPrefix two = "hashtwo";
   info->add_threat_entries()->set_hash(one);
   info->add_threat_entries()->set_hash(two);
 
+  std::unique_ptr<V4GetHashProtocolManager> pm(CreateProtocolManager());
+  req.mutable_client()->set_client_id(pm->config_.client_name);
+  req.mutable_client()->set_client_version(pm->config_.version);
+
   // Serialize and Base64 encode.
   std::string req_data, req_base64;
   req.SerializeToString(&req_data);
   base::Base64Encode(req_data, &req_base64);
 
-  std::unique_ptr<V4GetHashProtocolManager> pm(CreateProtocolManager());
+  std::vector<HashPrefix> prefixes_to_request = {one, two};
   EXPECT_EQ(req_base64, pm->GetHashRequest(prefixes_to_request));
 }
 
 TEST_F(V4GetHashProtocolManagerTest, TestParseHashResponse) {
   std::unique_ptr<V4GetHashProtocolManager> pm(CreateProtocolManager());
 
   base::Time now = base::Time::UnixEpoch();
   SetTestClock(now, pm.get());
 
   FullHash full_hash("Everything's shiny, Cap'n.");
@@ skipping 405 matching lines @@
   full_hash = FullHash("Everything's shiny, Cap'n.");
   info = ResponseInfo(full_hash, GetChromeUrlApiId());
   info.key_values.emplace_back("permission", "GEOLOCATION");
   infos.push_back(info);
   SetupFetcherToReturnOKResponse(factory, infos);
 
   EXPECT_TRUE(callback_called());
 }
 
 }  // namespace safe_browsing