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

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

Issue 60013005: Remove safe-browsing false-positive tracking. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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/safe_browsing_store_file_unittest.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_store_file_unittest.cc b/chrome/browser/safe_browsing/safe_browsing_store_file_unittest.cc
index 48f6d28a4583f3d7b717d89946ee7088e4787118..ac8002f5d6ef61084cfd036684e49a059382f4b0 100644
--- a/chrome/browser/safe_browsing/safe_browsing_store_file_unittest.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_store_file_unittest.cc
@@ -81,12 +81,10 @@ TEST_F(SafeBrowsingStoreFileTest, DetectsCorruption) {
// Can successfully open and read the store.
std::vector<SBAddFullHash> pending_adds;
- std::set<SBPrefix> prefix_misses;
SBAddPrefixes orig_prefixes;
std::vector<SBAddFullHash> orig_hashes;
EXPECT_TRUE(store_->BeginUpdate());
- EXPECT_TRUE(store_->FinishUpdate(pending_adds, prefix_misses,
- &orig_prefixes, &orig_hashes));
+ EXPECT_TRUE(store_->FinishUpdate(pending_adds, &orig_prefixes, &orig_hashes));
EXPECT_GT(orig_prefixes.size(), 0U);
EXPECT_GT(orig_hashes.size(), 0U);
EXPECT_FALSE(corruption_detected_);
@@ -108,8 +106,7 @@ TEST_F(SafeBrowsingStoreFileTest, DetectsCorruption) {
std::vector<SBAddFullHash> add_hashes;
corruption_detected_ = false;
EXPECT_TRUE(store_->BeginUpdate());
- EXPECT_FALSE(store_->FinishUpdate(pending_adds, prefix_misses,
- &add_prefixes, &add_hashes));
+ EXPECT_FALSE(store_->FinishUpdate(pending_adds, &add_prefixes, &add_hashes));
EXPECT_TRUE(corruption_detected_);
EXPECT_EQ(add_prefixes.size(), 0U);
EXPECT_EQ(add_hashes.size(), 0U);

Powered by Google App Engine
This is Rietveld 408576698