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

Unified Diff: chrome/browser/safe_browsing/safe_browsing_database_unittest.cc

Issue 14999008: Add a killswitch for CSD malware IP match and report feature. Use a new killswitch whitelist URL wh… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix the bug of 'and' -> && Created 7 years, 7 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/safe_browsing/safe_browsing_database_unittest.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_database_unittest.cc b/chrome/browser/safe_browsing/safe_browsing_database_unittest.cc
index ca616446c9d19d57c3c6054d92a8ada9d2844779..7d4168469b5289284803106e7362871bce0299d7 100644
--- a/chrome/browser/safe_browsing/safe_browsing_database_unittest.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_database_unittest.cc
@@ -1367,6 +1367,19 @@ TEST_F(SafeBrowsingDatabaseTest, Whitelists) {
EXPECT_FALSE(database_->ContainsDownloadWhitelistedUrl(
GURL(std::string("http://www.google.com/"))));
+ // Test only add the malware IP killswitch
+ csd_chunks.clear();
+ chunk.hosts.clear();
+ InsertAddChunkHostFullHashes(
+ &chunk, 15, "sb-ssl.google.com/",
+ "sb-ssl.google.com/safebrowsing/csd/killswitch_malware");
+ csd_chunks.push_back(chunk);
+ EXPECT_TRUE(database_->UpdateStarted(&lists));
+ database_->InsertChunks(safe_browsing_util::kCsdWhiteList, csd_chunks);
+ database_->UpdateFinished(true);
+
+ EXPECT_TRUE(database_->MalwareIPMatchKillSwitchOn());
+
// Test that the kill-switch works as intended.
csd_chunks.clear();
download_chunks.clear();
@@ -1375,7 +1388,6 @@ TEST_F(SafeBrowsingDatabaseTest, Whitelists) {
InsertAddChunkHostFullHashes(&chunk, 5, "sb-ssl.google.com/",
"sb-ssl.google.com/safebrowsing/csd/killswitch");
csd_chunks.push_back(chunk);
-
chunk.hosts.clear();
InsertAddChunkHostFullHashes(&chunk, 5, "sb-ssl.google.com/",
"sb-ssl.google.com/safebrowsing/csd/killswitch");
@@ -1387,6 +1399,7 @@ TEST_F(SafeBrowsingDatabaseTest, Whitelists) {
download_chunks);
database_->UpdateFinished(true);
+ EXPECT_TRUE(database_->MalwareIPMatchKillSwitchOn());
EXPECT_TRUE(database_->ContainsCsdWhitelistedUrl(
GURL(std::string("https://") + kGood1Url2 + "/c.html")));
EXPECT_TRUE(database_->ContainsCsdWhitelistedUrl(
@@ -1415,6 +1428,12 @@ TEST_F(SafeBrowsingDatabaseTest, Whitelists) {
csd_chunks.push_back(sub_chunk);
sub_chunk.hosts.clear();
+ InsertSubChunkHostFullHash(
+ &sub_chunk, 10, 15, "sb-ssl.google.com/",
+ "sb-ssl.google.com/safebrowsing/csd/killswitch_malware");
+ csd_chunks.push_back(sub_chunk);
+
+ sub_chunk.hosts.clear();
InsertSubChunkHostFullHash(&sub_chunk, 1, 5,
"sb-ssl.google.com/",
"sb-ssl.google.com/safebrowsing/csd/killswitch");
@@ -1426,6 +1445,7 @@ TEST_F(SafeBrowsingDatabaseTest, Whitelists) {
download_chunks);
database_->UpdateFinished(true);
+ EXPECT_FALSE(database_->MalwareIPMatchKillSwitchOn());
EXPECT_TRUE(database_->ContainsCsdWhitelistedUrl(
GURL(std::string("https://") + kGood1Url2 + "/c.html")));
EXPECT_TRUE(database_->ContainsCsdWhitelistedUrl(

Powered by Google App Engine
This is Rietveld 408576698