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

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

Issue 2743923002: Network traffic annotation added to safe_search_url_checker. (Closed)
Patch Set: Annotation updated. Created 3 years, 9 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
« no previous file with comments | « chrome/browser/safe_search_api/safe_search_url_checker_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/supervised_user/supervised_user_url_filter.cc
diff --git a/chrome/browser/supervised_user/supervised_user_url_filter.cc b/chrome/browser/supervised_user/supervised_user_url_filter.cc
index 5318b1bf28dbae99b0708f0830cd0f0ab649f8a2..93a3456b1dd8efc98bdb08448edbb9ddc5a5ba2e 100644
--- a/chrome/browser/supervised_user/supervised_user_url_filter.cc
+++ b/chrome/browser/supervised_user/supervised_user_url_filter.cc
@@ -531,7 +531,29 @@ void SupervisedUserURLFilter::SetManualURLs(
void SupervisedUserURLFilter::InitAsyncURLChecker(
net::URLRequestContextGetter* context) {
- async_url_checker_.reset(new SafeSearchURLChecker(context));
+ net::NetworkTrafficAnnotationTag traffic_annotation =
+ net::DefineNetworkTrafficAnnotation("supervised_user_url_filter", R"(
+ semantics {
+ sender: "Supervised Users"
+ description:
+ "Checks whether a given URL (or set of URLs) is considered safe by "
+ "Google SafeSearch."
+ trigger:
+ "If the parent enabled this feature for the child account, this is "
+ "sent for every navigation."
+ data: "URL(s) to be checked."
+ destination: GOOGLE_OWNED_SERVICE
+ }
+ policy {
+ cookies_allowed: false
+ setting:
+ "This feature is only used in child accounts and cannot be "
+ "disabled by settings. Parent accounts can disable it in the "
+ "family dashboard."
+ policy_exception_justification: "Not implemented."
+ })");
+ async_url_checker_.reset(
+ new SafeSearchURLChecker(context, traffic_annotation));
}
void SupervisedUserURLFilter::ClearAsyncURLChecker() {
« no previous file with comments | « chrome/browser/safe_search_api/safe_search_url_checker_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698