Remove a use of stl_util in safe browsing.

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..93dc2dd2d298ecf13ce5ff54210da2fd1f10a145 100644
--- a/components/safe_browsing_db/v4_get_hash_protocol_manager.h
+++ b/components/safe_browsing_db/v4_get_hash_protocol_manager.h
@@ -138,7 +138,10 @@ class V4GetHashProtocolManager : public net::URLFetcherDelegate,
 
  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.