Componentize safe_browsing: decouple threat_details* from the chrome/ layer.

chrome/browser/safe_browsing/threat_details.h

Index: chrome/browser/safe_browsing/threat_details.h
diff --git a/chrome/browser/safe_browsing/threat_details.h b/chrome/browser/safe_browsing/threat_details.h
index e5f6ac564a0e2bb60d20678f3a32f35427808deb..abd1747be15ae0510743831c61544e119a00d2f0 100644
--- a/chrome/browser/safe_browsing/threat_details.h
+++ b/chrome/browser/safe_browsing/threat_details.h
@@ -20,22 +20,27 @@
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
-#include "chrome/browser/safe_browsing/ui_manager.h"
 #include "components/safe_browsing/common/safebrowsing_types.h"
 #include "components/safe_browsing/csd.pb.h"
+#include "components/security_interstitials/content/unsafe_resource.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "net/base/completion_callback.h"
 
+namespace history {
+class HistoryService;
+}  // namespace history
+
 namespace net {
 class URLRequestContextGetter;
-}
+}  // namespace net
 
-class Profile;
 struct SafeBrowsingHostMsg_ThreatDOMDetails_Node;
 
 namespace safe_browsing {
 
+class BaseUIManager;
+
 // Maps a URL to its Resource.
 class ThreatDetailsCacheCollector;
 class ThreatDetailsRedirectsCollector;
@@ -72,11 +77,14 @@ class ThreatDetails : public base::RefCountedThreadSafe<
   typedef security_interstitials::UnsafeResource UnsafeResource;
 
   // Constructs a new ThreatDetails instance, using the factory.
-  static ThreatDetails* NewThreatDetails(BaseUIManager* ui_manager,
-                                         content::WebContents* web_contents,
-                                         const UnsafeResource& resource);
+  static ThreatDetails* NewThreatDetails(
+      BaseUIManager* ui_manager,
+      content::WebContents* web_contents,
+      const UnsafeResource& resource,
+      net::URLRequestContextGetter* request_context_getter,
+      history::HistoryService* history_service);
 
-  // Makes the passed |factory| the factory used to instanciate
+  // Makes the passed |factory| the factory used to instantiate
   // SafeBrowsingBlockingPage objects. Useful for tests.
   static void RegisterFactory(ThreatDetailsFactory* factory) {
     factory_ = factory;
@@ -104,7 +112,9 @@ class ThreatDetails : public base::RefCountedThreadSafe<
 
   ThreatDetails(BaseUIManager* ui_manager,
                 content::WebContents* web_contents,
-                const UnsafeResource& resource);
+                const UnsafeResource& resource,
+                net::URLRequestContextGetter* request_context_getter,
+                history::HistoryService* history_service);
 
   ~ThreatDetails() override;
 
@@ -114,8 +124,6 @@ class ThreatDetails : public base::RefCountedThreadSafe<
       const GURL& frame_last_committed_url,
       const std::vector<SafeBrowsingHostMsg_ThreatDOMDetails_Node>& params);
 
-  Profile* profile_;
-
   // The report protocol buffer.
   std::unique_ptr<ClientSafeBrowsingReportRequest> report_;
 
@@ -244,7 +252,9 @@ class ThreatDetailsFactory {
   virtual ThreatDetails* CreateThreatDetails(
       BaseUIManager* ui_manager,
       content::WebContents* web_contents,
-      const SafeBrowsingUIManager::UnsafeResource& unsafe_resource) = 0;
+      const security_interstitials::UnsafeResource& unsafe_resource,
+      net::URLRequestContextGetter* request_context_getter,
+      history::HistoryService* history_service) = 0;
 };
 
 }  // namespace safe_browsing