Set client_id and client_version in requests sent to SafeBrowsing

components/safe_browsing_db/v4_update_protocol_manager.h

Index: components/safe_browsing_db/v4_update_protocol_manager.h
diff --git a/components/safe_browsing_db/v4_update_protocol_manager.h b/components/safe_browsing_db/v4_update_protocol_manager.h
index a8c1624b82dfdf3fd0ba6e029f7d9fa13f01b416..e172d42451370e9f0cc2ee333de709f21dd1a5a9 100644
--- a/components/safe_browsing_db/v4_update_protocol_manager.h
+++ b/components/safe_browsing_db/v4_update_protocol_manager.h
@@ -100,7 +100,8 @@ class V4UpdateProtocolManager : public net::URLFetcherDelegate,
   // Fills a FetchThreatListUpdatesRequest protocol buffer for a request.
   // Returns the serialized and base64 URL encoded request as a string.
   static std::string GetBase64SerializedUpdateRequestProto(
-      const StoreStateMap& store_state_map);
+      const StoreStateMap& store_state_map,

[Nathan Parker, 2016/09/20 00:54:36]

Would it be cleaner to make this function non-static and use config_, rather
than passing it in?

+      const V4ProtocolConfig& config);
 
   // Parses the base64 encoded response received from the server as a
   // FetchThreatListUpdatesResponse protobuf and returns each of the