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

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

Issue 337723004: [Safe browsing] Clean up code to scan hash results for threats. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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_browsing/safe_browsing_util.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/safe_browsing/safe_browsing_util_unittest.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_util_unittest.cc b/chrome/browser/safe_browsing/safe_browsing_util_unittest.cc
index d12586fbc443d2e728e0ce375b00d650743e6025..8de290dcefe5fbf37a8c83b9ba364d013f58d987 100644
--- a/chrome/browser/safe_browsing/safe_browsing_util_unittest.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_util_unittest.cc
@@ -277,19 +277,6 @@ TEST(SafeBrowsingUtilTest, CanonicalizeUrl) {
}
}
-TEST(SafeBrowsingUtilTest, GetUrlHashIndex) {
- GURL url("http://www.evil.com/phish.html");
- SBFullHashResult full_hash;
- full_hash.hash = SBFullHashForString(url.host() + url.path());
- std::vector<SBFullHashResult> full_hashes;
- full_hashes.push_back(full_hash);
-
- EXPECT_EQ(safe_browsing_util::GetUrlHashIndex(url, full_hashes), 0);
-
- url = GURL("http://www.evil.com/okay_path.html");
- EXPECT_EQ(safe_browsing_util::GetUrlHashIndex(url, full_hashes), -1);
-}
-
TEST(SafeBrowsingUtilTest, ListIdListNameConversion) {
std::string list_name;
EXPECT_FALSE(safe_browsing_util::GetListName(safe_browsing_util::INVALID,
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_util.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698