PVer4: Re-use the PVer3 implementation to get full hashes

components/safe_browsing_db/v4_get_hash_protocol_manager.h

Index: components/safe_browsing_db/v4_get_hash_protocol_manager.h
diff --git a/components/safe_browsing_db/v4_get_hash_protocol_manager.h b/components/safe_browsing_db/v4_get_hash_protocol_manager.h
index c80f2da773f2d1489438d41e85060667c1b4f8f6..7cf003d925e70fbeb26da92016f76cd0fb478326 100644
--- a/components/safe_browsing_db/v4_get_hash_protocol_manager.h
+++ b/components/safe_browsing_db/v4_get_hash_protocol_manager.h
@@ -68,12 +68,21 @@ class V4GetHashProtocolManager : public net::URLFetcherDelegate,
 
   // Retrieve the full hash for a set of prefixes, and invoke the callback
   // argument when the results are retrieved. The callback may be invoked
-  // synchronously.
+  // synchronously. Works with V3 data structures.

[kcarattini, 2016/08/19 03:30:56]

Is this method going away eventually after the caching is moved? If so, can you
add that in the comment? If not, and these structures are going to be used with
V4 going forward, then we shouldn't call them V3 structures.

   virtual void GetFullHashes(const std::vector<SBPrefix>& prefixes,
                              const std::vector<PlatformType>& platforms,
                              ThreatType threat_type,
                              FullHashCallback callback);
 
+  // Retrieve the full hash for a set of prefixes, and invoke the callback
+  // argument when the results are retrieved. The callback may be invoked
+  // synchronously. Works with data types and structures for PVer4.
+  virtual void GetFullHashes(const base::hash_set<HashPrefix>& prefixes,
+                             const PlatformTypeSet& platforms,
+                             const ThreatEntryTypeSet& threat_entry_types,
+                             const ThreatTypeSet& threat_types,
+                             FullHashCallback callback);
+
   // Retrieve the full hash and API metadata for a set of prefixes, and invoke
   // the callback argument when the results are retrieved. The callback may be
   // invoked synchronously.