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

Unified Diff: chrome/browser/supervised_user/supervised_user_url_filter.h

Issue 2399823002: Extract the SafeSearch client to a separate directory (Closed)
Patch Set: Created 4 years, 2 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/supervised_user/supervised_user_url_filter.h
diff --git a/chrome/browser/supervised_user/supervised_user_url_filter.h b/chrome/browser/supervised_user/supervised_user_url_filter.h
index 7e5019338325143913604965efa4e9f0fac79dc0..e4e4397f46648b6d8f86ff96f22f7b6cf939ac05 100644
--- a/chrome/browser/supervised_user/supervised_user_url_filter.h
+++ b/chrome/browser/supervised_user/supervised_user_url_filter.h
@@ -16,6 +16,7 @@
#include "base/observer_list.h"
#include "base/threading/non_thread_safe.h"
#include "base/values.h"
+#include "chrome/browser/safe_search/safe_search_url_checker.h"
#include "chrome/browser/supervised_user/supervised_user_site_list.h"
#include "chrome/browser/supervised_user/supervised_users.h"
#include "components/supervised_user_error_page/supervised_user_error_page.h"
@@ -28,7 +29,7 @@ class URLRequestContextGetter;
}
class GURL;
-class SupervisedUserAsyncURLChecker;
+class SafeSearchURLChecker;
Marc Treib 2016/10/06 13:16:42 Not needed if you include the header anyway.
msramek 2016/10/06 15:08:41 Done.
// This class manages the filtering behavior for a given URL, i.e. it tells
// callers if a given URL should be allowed, blocked or warned about. It uses
@@ -185,7 +186,7 @@ class SupervisedUserURLFilter
void CheckCallback(const FilteringBehaviorCallback& callback,
const GURL& url,
- FilteringBehavior behavior,
+ SafeSearchURLChecker::Classification classification,
bool uncertain) const;
// This is mutable to allow notification in const member functions.
@@ -205,7 +206,7 @@ class SupervisedUserURLFilter
// Not owned.
const SupervisedUserBlacklist* blacklist_;
- std::unique_ptr<SupervisedUserAsyncURLChecker> async_url_checker_;
+ std::unique_ptr<SafeSearchURLChecker> async_url_checker_;
scoped_refptr<base::TaskRunner> blocking_task_runner_;

Powered by Google App Engine
This is Rietveld 408576698