Componentize safe_browsing: factor out chrome/ deps in ThreatDetailsRedirectsCollector.

chrome/browser/safe_browsing/threat_details_history.cc

 // Copyright (c) 2012 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.
 //
 // Implementation of the ThreatDetailsRedirectsCollector class.
 
 #include "chrome/browser/safe_browsing/threat_details_history.h"
 
 #include <stddef.h>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
-#include "chrome/browser/chrome_notification_types.h"
-#include "chrome/browser/history/history_service_factory.h"
-#include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/safe_browsing/threat_details.h"
+#include "components/history/core/browser/history_service.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_source.h"
 
 using content::BrowserThread;
 
 namespace safe_browsing {
 
 ThreatDetailsRedirectsCollector::ThreatDetailsRedirectsCollector(
-    Profile* profile)
-    : profile_(profile), has_started_(false) {
+    const base::WeakPtr<history::HistoryService>& history_service)
+    : has_started_(false),
+      history_service_(history_service),
+      history_service_observer_(this) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
-  if (profile) {
-    registrar_.Add(this, chrome::NOTIFICATION_PROFILE_DESTROYED,
-                   content::Source<Profile>(profile));
+
+  if (history_service) {
+    history_service_observer_.Add(history_service.get());
   }
 }
 
 void ThreatDetailsRedirectsCollector::StartHistoryCollection(
     const std::vector<GURL>& urls,
     const base::Closure& callback) {
   DVLOG(1) << "Num of urls to check in history service: " << urls.size();
   has_started_ = true;
   callback_ = callback;
 
@@ skipping 10 matching lines @@
 
 bool ThreatDetailsRedirectsCollector::HasStarted() const {
   return has_started_;
 }
 
 const std::vector<RedirectChain>&
 ThreatDetailsRedirectsCollector::GetCollectedUrls() const {
   return redirects_urls_;
 }
 
-void ThreatDetailsRedirectsCollector::Observe(
-    int type,
-    const content::NotificationSource& source,
-    const content::NotificationDetails& details) {
-  DCHECK_CURRENTLY_ON(BrowserThread::UI);
-  DCHECK_EQ(type, chrome::NOTIFICATION_PROFILE_DESTROYED);
-  DVLOG(1) << "Profile gone.";
-  profile_ = NULL;
-}
-
 ThreatDetailsRedirectsCollector::~ThreatDetailsRedirectsCollector() {}
 
 void ThreatDetailsRedirectsCollector::StartGetRedirects(
     const std::vector<GURL>& urls) {
   // History access from profile needs to happen in UI thread
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   for (size_t i = 0; i < urls.size(); ++i) {
     urls_.push_back(urls[i]);
   }
   urls_it_ = urls_.begin();
   GetRedirects(*urls_it_);
 }
 
 void ThreatDetailsRedirectsCollector::GetRedirects(const GURL& url) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
-  if (!profile_) {
+
+  if (!history_service_) {
     AllDone();
     return;
   }
 
-  history::HistoryService* history = HistoryServiceFactory::GetForProfile(
-      profile_, ServiceAccessType::EXPLICIT_ACCESS);
-  if (!history) {
-    AllDone();
-    return;
-  }
-
-  history->QueryRedirectsTo(
-      url, base::Bind(&ThreatDetailsRedirectsCollector::OnGotQueryRedirectsTo,
-                      base::Unretained(this), url),
+  history_service_->QueryRedirectsTo(
+      url,
+      base::Bind(&ThreatDetailsRedirectsCollector::OnGotQueryRedirectsTo,
+                 base::Unretained(this), url),
       &request_tracker_);
 }
 
 void ThreatDetailsRedirectsCollector::OnGotQueryRedirectsTo(
     const GURL& url,
     const history::RedirectList* redirect_list) {
   if (!redirect_list->empty()) {
     std::vector<GURL> urllist;
     urllist.push_back(url);
     urllist.insert(urllist.end(), redirect_list->begin(), redirect_list->end());
@@ skipping 10 matching lines @@
 
   GetRedirects(*urls_it_);
 }
 
 void ThreatDetailsRedirectsCollector::AllDone() {
   DVLOG(1) << "AllDone";
   BrowserThread::PostTask(BrowserThread::IO, FROM_HERE, callback_);
   callback_.Reset();
 }
 
+void ThreatDetailsRedirectsCollector::HistoryServiceBeingDeleted(
+    history::HistoryService* history_service) {
+  history_service_observer_.Remove(history_service);
+  history_service_.reset();
+}
+
 }  // namespace safe_browsing