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

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

Issue 1410343012: Revert of Move more declarations from c/b/sb/sb_util.h to components. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@01_components
Patch Set: Created 5 years, 1 month 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/local_database_manager_unittest.cc
diff --git a/chrome/browser/safe_browsing/local_database_manager_unittest.cc b/chrome/browser/safe_browsing/local_database_manager_unittest.cc
index b4bbc3056fe7bbef884f5caba63e3513bb7d0253..de8dc964612989ca866c12f6ba2683000136a343 100644
--- a/chrome/browser/safe_browsing/local_database_manager_unittest.cc
+++ b/chrome/browser/safe_browsing/local_database_manager_unittest.cc
@@ -40,7 +40,7 @@
class SafeBrowsingDatabaseManagerTest : public PlatformTest {
public:
- bool RunSBHashTest(const safe_browsing::ListType list_type,
+ bool RunSBHashTest(const safe_browsing_util::ListType list_type,
const std::vector<SBThreatType>& expected_threats,
const std::string& result_list);
@@ -49,7 +49,7 @@
};
bool SafeBrowsingDatabaseManagerTest::RunSBHashTest(
- const safe_browsing::ListType list_type,
+ const safe_browsing_util::ListType list_type,
const std::vector<SBThreatType>& expected_threats,
const std::string& result_list) {
scoped_refptr<SafeBrowsingService> sb_service_(
@@ -69,7 +69,7 @@
const SBFullHashResult full_hash_result = {
same_full_hash,
- safe_browsing::GetListId(result_list)
+ safe_browsing_util::GetListId(result_list)
};
std::vector<SBFullHashResult> fake_results(1, full_hash_result);
@@ -82,23 +82,23 @@
TEST_F(SafeBrowsingDatabaseManagerTest, CheckCorrespondsListType) {
std::vector<SBThreatType> malware_threat(1,
SB_THREAT_TYPE_BINARY_MALWARE_URL);
- EXPECT_FALSE(RunSBHashTest(safe_browsing::BINURL,
+ EXPECT_FALSE(RunSBHashTest(safe_browsing_util::BINURL,
malware_threat,
- safe_browsing::kMalwareList));
- EXPECT_TRUE(RunSBHashTest(safe_browsing::BINURL,
+ safe_browsing_util::kMalwareList));
+ EXPECT_TRUE(RunSBHashTest(safe_browsing_util::BINURL,
malware_threat,
- safe_browsing::kBinUrlList));
+ safe_browsing_util::kBinUrlList));
// Check for multiple threats
std::vector<SBThreatType> multiple_threats;
multiple_threats.push_back(SB_THREAT_TYPE_URL_MALWARE);
multiple_threats.push_back(SB_THREAT_TYPE_URL_PHISHING);
- EXPECT_FALSE(RunSBHashTest(safe_browsing::MALWARE,
+ EXPECT_FALSE(RunSBHashTest(safe_browsing_util::MALWARE,
multiple_threats,
- safe_browsing::kBinUrlList));
- EXPECT_TRUE(RunSBHashTest(safe_browsing::MALWARE,
+ safe_browsing_util::kBinUrlList));
+ EXPECT_TRUE(RunSBHashTest(safe_browsing_util::MALWARE,
multiple_threats,
- safe_browsing::kMalwareList));
+ safe_browsing_util::kMalwareList));
}
TEST_F(SafeBrowsingDatabaseManagerTest, GetUrlSeverestThreatType) {
@@ -111,35 +111,31 @@
"http://www.unwantedandmalware.com/page.html");
const GURL kSafeUrl("http://www.safe.com/page.html");
- const SBFullHash kMalwareHostHash =
- safe_browsing::SBFullHashForString("malware.com/");
- const SBFullHash kPhishingHostHash =
- safe_browsing::SBFullHashForString("phishing.com/");
- const SBFullHash kUnwantedHostHash =
- safe_browsing::SBFullHashForString("unwanted.com/");
+ const SBFullHash kMalwareHostHash = SBFullHashForString("malware.com/");
+ const SBFullHash kPhishingHostHash = SBFullHashForString("phishing.com/");
+ const SBFullHash kUnwantedHostHash = SBFullHashForString("unwanted.com/");
const SBFullHash kUnwantedAndMalwareHostHash =
- safe_browsing::SBFullHashForString("unwantedandmalware.com/");
- const SBFullHash kSafeHostHash =
- safe_browsing::SBFullHashForString("www.safe.com/");
+ SBFullHashForString("unwantedandmalware.com/");
+ const SBFullHash kSafeHostHash = SBFullHashForString("www.safe.com/");
{
SBFullHashResult full_hash;
full_hash.hash = kMalwareHostHash;
- full_hash.list_id = static_cast<int>(safe_browsing::MALWARE);
+ full_hash.list_id = static_cast<int>(safe_browsing_util::MALWARE);
full_hashes.push_back(full_hash);
}
{
SBFullHashResult full_hash;
full_hash.hash = kPhishingHostHash;
- full_hash.list_id = static_cast<int>(safe_browsing::PHISH);
+ full_hash.list_id = static_cast<int>(safe_browsing_util::PHISH);
full_hashes.push_back(full_hash);
}
{
SBFullHashResult full_hash;
full_hash.hash = kUnwantedHostHash;
- full_hash.list_id = static_cast<int>(safe_browsing::UNWANTEDURL);
+ full_hash.list_id = static_cast<int>(safe_browsing_util::UNWANTEDURL);
full_hashes.push_back(full_hash);
}
@@ -148,12 +144,13 @@
// kUnwantedAndMalwareHostHash.
SBFullHashResult full_hash_malware;
full_hash_malware.hash = kUnwantedAndMalwareHostHash;
- full_hash_malware.list_id = static_cast<int>(safe_browsing::MALWARE);
+ full_hash_malware.list_id = static_cast<int>(safe_browsing_util::MALWARE);
full_hashes.push_back(full_hash_malware);
SBFullHashResult full_hash_unwanted;
full_hash_unwanted.hash = kUnwantedAndMalwareHostHash;
- full_hash_unwanted.list_id = static_cast<int>(safe_browsing::UNWANTEDURL);
+ full_hash_unwanted.list_id =
+ static_cast<int>(safe_browsing_util::UNWANTEDURL);
full_hashes.push_back(full_hash_unwanted);
}
« no previous file with comments | « chrome/browser/safe_browsing/local_database_manager.cc ('k') | chrome/browser/safe_browsing/protocol_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698