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

Unified Diff: components/subresource_filter/core/common/url_pattern_matching_unittest.cc

Issue 2183883002: Implement deactivating SubresourceFilter for document. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@integrate_ruleset
Patch Set: Address more comments from engedy@ Created 4 years, 4 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/core/common/url_pattern_matching_unittest.cc
diff --git a/components/subresource_filter/core/common/url_pattern_matching_unittest.cc b/components/subresource_filter/core/common/url_pattern_matching_unittest.cc
index 801baa66c6df8e378e7928c81ec5ce85f0f46cac..cdf36add41c8a3940af487d4497fbf9b56ffcf23 100644
--- a/components/subresource_filter/core/common/url_pattern_matching_unittest.cc
+++ b/components/subresource_filter/core/common/url_pattern_matching_unittest.cc
@@ -60,7 +60,7 @@ TEST(UrlPatternMatchingTest, BuildFailureFunctionForUrlPattern) {
}
}
-TEST(UrlPatternMatchingTest, IsMatch) {
+TEST(UrlPatternMatchingTest, IsUrlPatternMatch) {
const struct {
UrlPattern url_pattern;
const char* url;
@@ -142,8 +142,9 @@ TEST(UrlPatternMatchingTest, IsMatch) {
std::vector<size_t> failure;
BuildFailureFunction(test_case.url_pattern, &failure);
- const bool is_match = IsMatch(GURL(test_case.url), test_case.url_pattern,
- failure.begin(), failure.end());
+ const bool is_match =
+ IsUrlPatternMatch(GURL(test_case.url), test_case.url_pattern,
+ failure.begin(), failure.end());
EXPECT_EQ(test_case.expect_match, is_match);
}
}
« no previous file with comments | « components/subresource_filter/core/common/url_pattern_matching.h ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698