Index: chrome/browser/safe_browsing/safe_browsing_store_unittest.cc |
diff --git a/chrome/browser/safe_browsing/safe_browsing_store_unittest.cc b/chrome/browser/safe_browsing/safe_browsing_store_unittest.cc |
index 2c336abee300a82e7e735fd5c3705295a779b1c2..047510577cadfd6d3a4ca93877af2dc48e5fafbd 100644 |
--- a/chrome/browser/safe_browsing/safe_browsing_store_unittest.cc |
+++ b/chrome/browser/safe_browsing/safe_browsing_store_unittest.cc |
@@ -300,67 +300,4 @@ TEST(SafeBrowsingStoreTest, Y2K38) { |
<< " (int32)time_t is running out."; |
} |
-// Test that prefixes which were injected from full hashes are being removed. |
-// This was a mistake in earlier versions of the code. |
-TEST(SafeBrowsingStoreTest, KnockoutPrefixVolunteers) { |
- // Construct some full hashes which share prefix with another. |
- const SBFullHash kHash1mod1 = ModifyHashAfterPrefix(kHash1, 1); |
- const SBFullHash kHash2mod1 = ModifyHashAfterPrefix(kHash2, 1); |
- |
- SBAddPrefixes add_prefixes; |
- std::vector<SBAddFullHash> add_hashes; |
- SBSubPrefixes sub_prefixes; |
- std::vector<SBSubFullHash> sub_hashes; |
- |
- // Full hashes for an add chunk will have had the prefix injected. |
- add_hashes.push_back(SBAddFullHash(kAddChunk1, kHash1)); |
- add_hashes.push_back(SBAddFullHash(kAddChunk1, kHash1mod1)); |
- add_prefixes.push_back(SBAddPrefix(kAddChunk1, kHash1.prefix)); |
- |
- // Other full hashes or prefixes are not affected. |
- add_hashes.push_back(SBAddFullHash(kAddChunk1, kHash3)); |
- add_prefixes.push_back(SBAddPrefix(kAddChunk2, kHash4.prefix)); |
- |
- // Full hashes for a sub chunk will have had the prefix injected. |
- sub_hashes.push_back(SBSubFullHash(kSubChunk1, kAddChunk1, kHash2)); |
- sub_hashes.push_back(SBSubFullHash(kSubChunk1, kAddChunk1, kHash2mod1)); |
- sub_prefixes.push_back(SBSubPrefix(kSubChunk1, kAddChunk1, kHash2.prefix)); |
- |
- // Other full hashes or prefixes are not affected. |
- sub_hashes.push_back(SBSubFullHash(kSubChunk1, kAddChunk1, kHash5)); |
- sub_prefixes.push_back(SBSubPrefix(kSubChunk2, kAddChunk2, kHash6.prefix)); |
- |
- const base::hash_set<int32> no_deletions; |
- ProcessHelper(&add_prefixes, &sub_prefixes, &add_hashes, &sub_hashes, |
- no_deletions, no_deletions); |
- |
- ASSERT_EQ(1U, add_prefixes.size()); |
- EXPECT_EQ(kAddChunk2, add_prefixes[0].chunk_id); |
- EXPECT_EQ(kHash4.prefix, add_prefixes[0].prefix); |
- |
- ASSERT_EQ(3U, add_hashes.size()); |
- EXPECT_EQ(kAddChunk1, add_hashes[0].chunk_id); |
- EXPECT_TRUE(SBFullHashEqual(kHash1mod1, add_hashes[0].full_hash)); |
- EXPECT_EQ(kAddChunk1, add_hashes[1].chunk_id); |
- EXPECT_TRUE(SBFullHashEqual(kHash1, add_hashes[1].full_hash)); |
- EXPECT_EQ(kAddChunk1, add_hashes[2].chunk_id); |
- EXPECT_TRUE(SBFullHashEqual(kHash3, add_hashes[2].full_hash)); |
- |
- ASSERT_EQ(1U, sub_prefixes.size()); |
- EXPECT_EQ(kSubChunk2, sub_prefixes[0].chunk_id); |
- EXPECT_EQ(kAddChunk2, sub_prefixes[0].add_chunk_id); |
- EXPECT_EQ(kHash6.prefix, sub_prefixes[0].add_prefix); |
- |
- ASSERT_EQ(3U, sub_hashes.size()); |
- EXPECT_EQ(kSubChunk1, sub_hashes[0].chunk_id); |
- EXPECT_EQ(kAddChunk1, sub_hashes[0].add_chunk_id); |
- EXPECT_TRUE(SBFullHashEqual(kHash5, sub_hashes[0].full_hash)); |
- EXPECT_EQ(kSubChunk1, sub_hashes[1].chunk_id); |
- EXPECT_EQ(kAddChunk1, sub_hashes[1].add_chunk_id); |
- EXPECT_TRUE(SBFullHashEqual(kHash2mod1, sub_hashes[1].full_hash)); |
- EXPECT_EQ(kSubChunk1, sub_hashes[2].chunk_id); |
- EXPECT_EQ(kAddChunk1, sub_hashes[2].add_chunk_id); |
- EXPECT_TRUE(SBFullHashEqual(kHash2, sub_hashes[2].full_hash)); |
-} |
- |
} // namespace |