Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5526)

Unified Diff: chrome/browser/safe_browsing/threat_details_history.h

Issue 2836103002: Componentize safe_browsing: move threat_details* to component. (Closed)
Patch Set: fix spelling and typos Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/threat_details_history.h
diff --git a/chrome/browser/safe_browsing/threat_details_history.h b/chrome/browser/safe_browsing/threat_details_history.h
deleted file mode 100644
index 70c1236031c002ff2f34cbe5187db559c9b9ff00..0000000000000000000000000000000000000000
--- a/chrome/browser/safe_browsing/threat_details_history.h
+++ /dev/null
@@ -1,94 +0,0 @@
-// 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.
-
-#ifndef CHROME_BROWSER_SAFE_BROWSING_THREAT_DETAILS_HISTORY_H_
-#define CHROME_BROWSER_SAFE_BROWSING_THREAT_DETAILS_HISTORY_H_
-
-// This class gets redirect chain for urls from the history service.
-
-#include <string>
-#include <vector>
-
-#include "base/callback.h"
-#include "base/containers/hash_tables.h"
-#include "base/macros.h"
-#include "base/memory/ref_counted.h"
-#include "base/scoped_observer.h"
-#include "base/sequenced_task_runner_helpers.h"
-#include "base/task/cancelable_task_tracker.h"
-#include "components/history/core/browser/history_service_observer.h"
-#include "content/public/browser/browser_thread.h"
-#include "net/base/completion_callback.h"
-
-namespace safe_browsing {
-
-typedef std::vector<GURL> RedirectChain;
-
-class ThreatDetailsRedirectsCollector
- : public base::RefCountedThreadSafe<
- ThreatDetailsRedirectsCollector,
- content::BrowserThread::DeleteOnUIThread>,
- public history::HistoryServiceObserver {
- public:
- explicit ThreatDetailsRedirectsCollector(
- const base::WeakPtr<history::HistoryService>& history_service);
-
- // Collects urls' redirects chain information from the history service.
- // We get access to history service via web_contents in UI thread.
- // Notice the callback will be posted to the IO thread.
- void StartHistoryCollection(const std::vector<GURL>& urls,
- const base::Closure& callback);
-
- // Returns whether or not StartCacheCollection has been called.
- bool HasStarted() const;
-
- // Returns the redirect urls we get from history service
- const std::vector<RedirectChain>& GetCollectedUrls() const;
-
- // history::HistoryServiceObserver
- void HistoryServiceBeingDeleted(
- history::HistoryService* history_service) override;
-
- private:
- friend struct content::BrowserThread::DeleteOnThread<
- content::BrowserThread::UI>;
- friend class base::DeleteHelper<ThreatDetailsRedirectsCollector>;
-
- ~ThreatDetailsRedirectsCollector() override;
-
- void StartGetRedirects(const std::vector<GURL>& urls);
- void GetRedirects(const GURL& url);
- void OnGotQueryRedirectsTo(const GURL& url,
- const history::RedirectList* redirect_list);
-
- // Posts the callback method back to IO thread when redirects collecting
- // is all done.
- void AllDone();
-
- base::CancelableTaskTracker request_tracker_;
-
- // Method we call when we are done. The caller must be alive for the
- // whole time, we are modifying its state (see above).
- base::Closure callback_;
-
- // Sets to true once StartHistoryCollection is called
- bool has_started_;
-
- // The urls we need to get redirects for
- std::vector<GURL> urls_;
- // The iterator goes over urls_
- std::vector<GURL>::iterator urls_it_;
- // The collected directs from history service
- std::vector<RedirectChain> redirects_urls_;
-
- base::WeakPtr<history::HistoryService> history_service_;
- ScopedObserver<history::HistoryService, history::HistoryServiceObserver>
- history_service_observer_;
-
- DISALLOW_COPY_AND_ASSIGN(ThreatDetailsRedirectsCollector);
-};
-
-} // namespace safe_browsing
-
-#endif // CHROME_BROWSER_SAFE_BROWSING_THREAT_DETAILS_HISTORY_H_
« no previous file with comments | « chrome/browser/safe_browsing/threat_details_cache.cc ('k') | chrome/browser/safe_browsing/threat_details_history.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698