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

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

Issue 2239753002: Added a ForceYouTubeRestrict policy and deprecated the old ForceYouTubeSafetyMode policy (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and inc'ed policy ID to 348 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/net/safe_search_util_unittest.cc
diff --git a/chrome/browser/net/safe_search_util_unittest.cc b/chrome/browser/net/safe_search_util_unittest.cc
index f5e66ce9ad6fa7e183ac5c09705e5d62ece65a00..0a4fb70204ef4908244d8069032e6706494d85c4 100644
--- a/chrome/browser/net/safe_search_util_unittest.cc
+++ b/chrome/browser/net/safe_search_util_unittest.cc
@@ -143,28 +143,50 @@ TEST_F(SafeSearchUtilTest, AddGoogleSafeSearchParams) {
TEST_F(SafeSearchUtilTest, SetYoutubeHeader) {
std::unique_ptr<net::URLRequest> request = CreateYoutubeRequest();
net::HttpRequestHeaders headers;
- safe_search_util::ForceYouTubeSafetyMode(request.get(), &headers);
+ safe_search_util::ForceYouTubeRestrict(
+ request.get(), &headers, safe_search_util::YOUTUBE_RESTRICT_MODERATE);
std::string value;
- EXPECT_TRUE(headers.GetHeader("Youtube-Safety-Mode", &value));
- EXPECT_EQ("Active", value);
+ EXPECT_TRUE(headers.GetHeader("Youtube-Restrict", &value));
+ EXPECT_EQ("Moderate", value);
}
TEST_F(SafeSearchUtilTest, OverrideYoutubeHeader) {
std::unique_ptr<net::URLRequest> request = CreateYoutubeRequest();
net::HttpRequestHeaders headers;
- headers.SetHeader("Youtube-Safety-Mode", "Off");
- safe_search_util::ForceYouTubeSafetyMode(request.get(), &headers);
+ headers.SetHeader("Youtube-Restrict", "Off");
+ safe_search_util::ForceYouTubeRestrict(
+ request.get(), &headers, safe_search_util::YOUTUBE_RESTRICT_MODERATE);
std::string value;
- EXPECT_TRUE(headers.GetHeader("Youtube-Safety-Mode", &value));
- EXPECT_EQ("Active", value);
+ EXPECT_TRUE(headers.GetHeader("Youtube-Restrict", &value));
+ EXPECT_EQ("Moderate", value);
}
TEST_F(SafeSearchUtilTest, DoesntTouchNonYoutubeURL) {
std::unique_ptr<net::URLRequest> request = CreateNonYoutubeRequest();
net::HttpRequestHeaders headers;
- headers.SetHeader("Youtube-Safety-Mode", "Off");
- safe_search_util::ForceYouTubeSafetyMode(request.get(), &headers);
+ headers.SetHeader("Youtube-Restrict", "Off");
+ safe_search_util::ForceYouTubeRestrict(
+ request.get(), &headers, safe_search_util::YOUTUBE_RESTRICT_MODERATE);
std::string value;
- EXPECT_TRUE(headers.GetHeader("Youtube-Safety-Mode", &value));
+ EXPECT_TRUE(headers.GetHeader("Youtube-Restrict", &value));
EXPECT_EQ("Off", value);
}
+
+TEST_F(SafeSearchUtilTest, SettingOffReturnsIfNothingIsSet) {
+ std::unique_ptr<net::URLRequest> request = CreateYoutubeRequest();
+ net::HttpRequestHeaders headers;
+ safe_search_util::ForceYouTubeRestrict(
+ request.get(), &headers, safe_search_util::YOUTUBE_RESTRICT_OFF);
+ std::string value;
+ EXPECT_FALSE(headers.GetHeader("Youtube-Restrict", &value));
+}
+
+TEST_F(SafeSearchUtilTest, SettingOffWipesIfSomethingIsSet) {
+ std::unique_ptr<net::URLRequest> request = CreateYoutubeRequest();
+ net::HttpRequestHeaders headers;
+ headers.SetHeader("Youtube-Restrict", "Moderate");
+ safe_search_util::ForceYouTubeRestrict(
+ request.get(), &headers, safe_search_util::YOUTUBE_RESTRICT_OFF);
+ std::string value;
+ EXPECT_FALSE(headers.GetHeader("Youtube-Restrict", &value));
+}

Powered by Google App Engine
This is Rietveld 408576698