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

Unified Diff: components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc

Issue 2645283007: Add the client for accessing Subresource Filter only list. (Closed)
Patch Set: Created 3 years, 11 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: components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc
diff --git a/components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc b/components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc
index 75db9c7231abce97ea59d332b134d735760f2389..e5c711e99a5a043f9ea1e18cd9fe1352fa294f75 100644
--- a/components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc
+++ b/components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc
@@ -27,15 +27,17 @@ namespace {
const char kExampleUrlWithParams[] = "https://example.com/soceng?q=engsoc";
const char kExampleUrl[] = "https://example.com";
const char kExampleLoginUrl[] = "https://example.com/login";
-const char kMatchesPatternHistogramName[] =
- "SubresourceFilter.PageLoad.RedirectChainMatchPattern";
-const char kNavigationChainSize[] =
- "SubresourceFilter.PageLoad.RedirectChainLength";
const char kUrlA[] = "https://example_a.com";
const char kUrlB[] = "https://example_b.com";
const char kUrlC[] = "https://example_c.com";
const char kUrlD[] = "https://example_d.com";
+const char kMatchesPatternHistogramName[] =
+ "SubresourceFilter.PageLoad.RedirectChainMatchPattern";
+const char kNavigationChainSize[] =
+ "SubresourceFilter.PageLoad.RedirectChainLength";
+const char kSubresourceFilterOnlySuffix[] = ".SubresourceFilterOnly";
+
// Human readable representation of expected redirect chain match patterns.
// The explanations for the buckets given for the following redirect chain:
// A->B->C->D, where A is initial URL and D is a final URL.
@@ -211,6 +213,7 @@ class ContentSubresourceFilterDriverFactoryTest
safe_browsing::SBThreatType threat_type,
safe_browsing::ThreatPatternType threat_type_metadata,
RedirectChainMatchPattern extected_pattern,
+ const std::string histogram_suffix,
bool expected_activation) {
base::HistogramTester tester;
EXPECT_CALL(*client(), ToggleNotificationVisibility(false)).Times(1);
@@ -246,16 +249,18 @@ class ContentSubresourceFilterDriverFactoryTest
::testing::Mock::VerifyAndClearExpectations(driver());
if (extected_pattern != EMPTY) {
- EXPECT_THAT(tester.GetAllSamples(kMatchesPatternHistogramName),
- ::testing::ElementsAre(base::Bucket(extected_pattern, 1)));
EXPECT_THAT(
- tester.GetAllSamples(kNavigationChainSize),
+ tester.GetAllSamples(kMatchesPatternHistogramName + histogram_suffix),
+ ::testing::ElementsAre(base::Bucket(extected_pattern, 1)));
+ EXPECT_THAT(
+ tester.GetAllSamples(kNavigationChainSize + histogram_suffix),
::testing::ElementsAre(base::Bucket(navigation_chain.size(), 1)));
} else {
- EXPECT_THAT(tester.GetAllSamples(kMatchesPatternHistogramName),
- ::testing::IsEmpty());
- EXPECT_THAT(tester.GetAllSamples(kNavigationChainSize),
+ EXPECT_THAT(
+ tester.GetAllSamples(kMatchesPatternHistogramName + histogram_suffix),
+ ::testing::IsEmpty());
+ EXPECT_THAT(tester.GetAllSamples(kNavigationChainSize + histogram_suffix),
::testing::IsEmpty());
}
}
@@ -278,10 +283,11 @@ class ContentSubresourceFilterDriverFactoryTest
safe_browsing::SBThreatType threat_type,
safe_browsing::ThreatPatternType threat_type_metadata,
RedirectChainMatchPattern extected_pattern,
+ const std::string& histogram_suffix,
bool expected_activation) {
BlacklistURLWithRedirectsNavigateAndCommit(
blacklisted_urls, navigation_chain, threat_type, threat_type_metadata,
- extected_pattern, expected_activation);
+ extected_pattern, histogram_suffix, expected_activation);
NavigateAndCommitSubframe(GURL(kExampleLoginUrl), expected_activation);
}
@@ -294,7 +300,7 @@ class ContentSubresourceFilterDriverFactoryTest
blacklisted_urls, navigation_chain,
safe_browsing::SB_THREAT_TYPE_URL_PHISHING,
safe_browsing::ThreatPatternType::SOCIAL_ENGINEERING_ADS,
- extected_pattern, expected_activation);
+ extected_pattern, std::string(), expected_activation);
}
void EmulateInPageNavigation(const std::vector<bool>& blacklisted_urls,
@@ -452,7 +458,6 @@ TEST_F(ContentSubresourceFilterDriverFactoryTest, RedirectPatternTest) {
{{false, true}, {GURL(kUrlA), GURL(kUrlB)}, F0M0L1, true},
{{true, false}, {GURL(kUrlA), GURL(kUrlB)}, F1M0L0, false},
{{true, true}, {GURL(kUrlA), GURL(kUrlB)}, F1M0L1, true},
-
{{false, false, false},
{GURL(kUrlA), GURL(kUrlB), GURL(kUrlC)},
EMPTY,
@@ -498,9 +503,13 @@ TEST_F(ContentSubresourceFilterDriverFactoryTest, RedirectPatternTest) {
test_data.blacklisted_urls, test_data.navigation_chain,
safe_browsing::SB_THREAT_TYPE_URL_PHISHING,
safe_browsing::ThreatPatternType::SOCIAL_ENGINEERING_ADS,
- test_data.hit_extected_pattern, test_data.expected_activation);
- NavigateAndExpectActivation({false}, {GURL("https://dummy.com")}, EMPTY,
- false);
+ test_data.hit_extected_pattern, std::string(),
+ test_data.expected_activation);
+ NavigateAndExpectActivation(
+ test_data.blacklisted_urls, test_data.navigation_chain,
+ safe_browsing::SB_THREAT_TYPE_SUBRESOURCE_FILTER_ONLY,
+ safe_browsing::ThreatPatternType::NONE, test_data.hit_extected_pattern,
+ kSubresourceFilterOnlySuffix, test_data.expected_activation);
}
}
@@ -540,7 +549,7 @@ TEST_P(ContentSubresourceFilterDriverFactoryThreatTypeTest,
test_data.threat_type,
test_data.threat_type_metadata,
test_data.expected_activation ? F0M0L1 : EMPTY,
- test_data.expected_activation);
+ std::string(), test_data.expected_activation);
};
TEST_P(ContentSubresourceFilterDriverFactoryActivationScopeTest,

Powered by Google App Engine
This is Rietveld 408576698