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

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

Issue 2401733002: Revert "Added a ForceYouTubeRestrict policy and deprecated the old ForceYouTubeSafetyMode policy" (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
« no previous file with comments | « no previous file | chrome/browser/net/chrome_network_delegate.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/chrome_network_delegate.h
diff --git a/chrome/browser/net/chrome_network_delegate.h b/chrome/browser/net/chrome_network_delegate.h
index 1252bb15c267a03f2c47228fd780993b56136983..bce63311264116e87fe81896cbc383a02db8dd6d 100644
--- a/chrome/browser/net/chrome_network_delegate.h
+++ b/chrome/browser/net/chrome_network_delegate.h
@@ -16,7 +16,6 @@
#include "base/memory/ref_counted.h"
#include "base/values.h"
#include "build/build_config.h"
-#include "chrome/browser/net/safe_search_util.h"
#include "components/data_use_measurement/content/data_use_measurement.h"
#include "components/metrics/data_use_tracker.h"
#include "components/prefs/pref_member.h"
@@ -79,8 +78,8 @@ class ChromeNetworkDelegate : public net::NetworkDelegateImpl {
url_blacklist_manager_ = url_blacklist_manager;
}
- // If |profile| is nullptr or not set, events will be broadcast to all
- // profiles, otherwise they will only be sent to the specified profile.
+ // If |profile| is NULL or not set, events will be broadcast to all profiles,
+ // otherwise they will only be sent to the specified profile.
// Also pass through to ChromeExtensionsNetworkDelegate::set_profile().
void set_profile(void* profile);
@@ -91,7 +90,7 @@ class ChromeNetworkDelegate : public net::NetworkDelegateImpl {
profile_path_ = profile_path;
}
- // If |cookie_settings| is nullptr or not set, all cookies are enabled,
+ // If |cookie_settings| is NULL or not set, all cookies are enabled,
// otherwise the settings are enforced on all observed network requests.
// Not inlined because we assign a scoped_refptr, which requires us to include
// the header file. Here we just forward-declare it.
@@ -106,9 +105,9 @@ class ChromeNetworkDelegate : public net::NetworkDelegateImpl {
force_google_safe_search_ = force_google_safe_search;
}
- void set_force_youtube_restrict(
- IntegerPrefMember* force_youtube_restrict) {
- force_youtube_restrict_ = force_youtube_restrict;
+ void set_force_youtube_safety_mode(
+ BooleanPrefMember* force_youtube_safety_mode) {
+ force_youtube_safety_mode_ = force_youtube_safety_mode;
}
void set_allowed_domains_for_apps(
@@ -126,13 +125,13 @@ class ChromeNetworkDelegate : public net::NetworkDelegateImpl {
bool is_data_usage_off_the_record);
// Binds the pref members to |pref_service| and moves them to the IO thread.
- // |enable_referrers| cannot be nullptr, the others can.
+ // |enable_referrers| cannot be NULL, the others can.
// This method should be called on the UI thread.
static void InitializePrefsOnUIThread(
BooleanPrefMember* enable_referrers,
BooleanPrefMember* enable_do_not_track,
BooleanPrefMember* force_google_safe_search,
- IntegerPrefMember* force_youtube_restrict,
+ BooleanPrefMember* force_youtube_safety_mode,
StringPrefMember* allowed_domains_for_apps,
PrefService* pref_service);
@@ -204,7 +203,7 @@ class ChromeNetworkDelegate : public net::NetworkDelegateImpl {
BooleanPrefMember* enable_referrers_;
BooleanPrefMember* enable_do_not_track_;
BooleanPrefMember* force_google_safe_search_;
- IntegerPrefMember* force_youtube_restrict_;
+ BooleanPrefMember* force_youtube_safety_mode_;
StringPrefMember* allowed_domains_for_apps_;
// Weak, owned by our owner.
« no previous file with comments | « no previous file | chrome/browser/net/chrome_network_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698