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

Unified Diff: chrome/browser/net/chrome_network_delegate.cc

Issue 1056003003: Policy: Ignore ForceSafeSearch if ForceGoogleSafeSearch or ForceYoutubeSafetyMode are enabled (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix Android Created 5 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/net/chrome_network_delegate.cc
diff --git a/chrome/browser/net/chrome_network_delegate.cc b/chrome/browser/net/chrome_network_delegate.cc
index 40d2687fa5e3b060eec6de4fa95c45ad1a22fa6f..27ff3ef2ea99cef5b7816787653a03960415f2cb 100644
--- a/chrome/browser/net/chrome_network_delegate.cc
+++ b/chrome/browser/net/chrome_network_delegate.cc
@@ -285,7 +285,6 @@ ChromeNetworkDelegate::ChromeNetworkDelegate(
: profile_(NULL),
enable_referrers_(enable_referrers),
enable_do_not_track_(NULL),
- force_safe_search_(NULL),
force_google_safe_search_(NULL),
force_youtube_safety_mode_(NULL),
#if defined(ENABLE_CONFIGURATION_POLICY)
@@ -334,7 +333,6 @@ void ChromeNetworkDelegate::NeverThrottleRequests() {
void ChromeNetworkDelegate::InitializePrefsOnUIThread(
BooleanPrefMember* enable_referrers,
BooleanPrefMember* enable_do_not_track,
- BooleanPrefMember* force_safe_search,
BooleanPrefMember* force_google_safe_search,
BooleanPrefMember* force_youtube_safety_mode,
PrefService* pref_service) {
@@ -347,11 +345,6 @@ void ChromeNetworkDelegate::InitializePrefsOnUIThread(
enable_do_not_track->MoveToThread(
BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO));
}
- if (force_safe_search) {
- force_safe_search->Init(prefs::kForceSafeSearch, pref_service);
- force_safe_search->MoveToThread(
- BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO));
- }
if (force_google_safe_search) {
force_google_safe_search->Init(prefs::kForceGoogleSafeSearch, pref_service);
force_google_safe_search->MoveToThread(
@@ -398,7 +391,6 @@ int ChromeNetworkDelegate::OnBeforeURLRequest(
request->SetExtraRequestHeaderByName(kDNTHeader, "1", true /* override */);
bool force_safe_search =
- (force_safe_search_ && force_safe_search_->GetValue()) ||
(force_google_safe_search_ && force_google_safe_search_->GetValue());
net::CompletionCallback wrapped_callback = callback;
@@ -425,10 +417,7 @@ int ChromeNetworkDelegate::OnBeforeSendHeaders(
net::URLRequest* request,
const net::CompletionCallback& callback,
net::HttpRequestHeaders* headers) {
- bool force_safety_mode =
- (force_safe_search_ && force_safe_search_->GetValue()) ||
- (force_youtube_safety_mode_ && force_youtube_safety_mode_->GetValue());
- if (force_safety_mode)
+ if (force_youtube_safety_mode_ && force_youtube_safety_mode_->GetValue())
safe_search_util::ForceYouTubeSafetyMode(request, headers);
return extensions_delegate_->OnBeforeSendHeaders(request, callback, headers);
« no previous file with comments | « chrome/browser/net/chrome_network_delegate.h ('k') | chrome/browser/net/chrome_network_delegate_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698