Remove a use of stl_util in safe browsing.

components/safe_browsing_db/v4_get_hash_protocol_manager.h

 // 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.
 
 #ifndef COMPONENTS_SAFE_BROWSING_DB_V4_GET_HASH_PROTOCOL_MANAGER_H_
 #define COMPONENTS_SAFE_BROWSING_DB_V4_GET_HASH_PROTOCOL_MANAGER_H_
 
 // A class that implements Chrome's interface with the SafeBrowsing V4 protocol.
 //
 // The V4GetHashProtocolManager handles formatting and making requests of, and
@@ skipping 120 matching lines @@
 
   // Resets the gethash error counter and multiplier.
   void ResetGetHashErrors();
 
   // Updates internal state for each GetHash response error, assuming that
   // the current time is |now|.
   void HandleGetHashError(const base::Time& now);
 
  private:
   // Map of GetHash requests to parameters which created it.
-  typedef base::hash_map<const net::URLFetcher*, FullHashCallback> HashRequests;
+  struct FetcherAndCallback;
+  typedef base::hash_map<const net::URLFetcher*,

[vabr (Chromium), 2016/08/29 09:14:50]

drive-by nit:
https://chromium-cpp.appspot.com/ prefers 'using new_alias = typename' to
'typedef typename new_alias'.

[Avi (use Gerrit), 2016/08/29 18:04:24]

Done.

+                         std::unique_ptr<FetcherAndCallback>>

[mattm, 2016/08/27 01:43:25]

Is the unique_ptr here so FetcherAndCallback can be forward declared?
What do you think about just using std::pair instead of the FetcherAndCallback
struct?

[Avi (use Gerrit), 2016/08/29 18:04:24]

I actually don't remember why I did it that specific way.
Not a huge fan of the whole "first"/"second" thing, but it would probably be
simpler.

+      HashRequests;
 
   // The factory that controls the creation of V4GetHashProtocolManager.
   // This is used by tests.
   static V4GetHashProtocolManagerFactory* factory_;
 
   // Current active request (in case we need to cancel) for updates or chunks
   // from the SafeBrowsing service. We can only have one of these outstanding
   // at any given time unlike GetHash requests, which are tracked separately.
   std::unique_ptr<net::URLFetcher> request_;
 
@@ skipping 35 matching lines @@
       net::URLRequestContextGetter* request_context_getter,
       const V4ProtocolConfig& config) = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(V4GetHashProtocolManagerFactory);
 };
 
 }  // namespace safe_browsing
 
 #endif  // COMPONENTS_SAFE_BROWSING_DB_V4_GET_HASH_PROTOCOL_MANAGER_H_