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

Unified Diff: components/subresource_filter/core/common/test_ruleset_creator.h

Issue 2661433002: Introduce VerifiedRulesetDealer and its async Handle. (Closed)
Patch Set: Refactor tests. 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/core/common/test_ruleset_creator.h
diff --git a/components/subresource_filter/core/common/test_ruleset_creator.h b/components/subresource_filter/core/common/test_ruleset_creator.h
index eef00b0e56f592055381fed407114a9a5c6377bc..9ff0b53e2eeb6fada4f806afede6584c68d0d694 100644
--- a/components/subresource_filter/core/common/test_ruleset_creator.h
+++ b/components/subresource_filter/core/common/test_ruleset_creator.h
@@ -30,6 +30,15 @@ struct TestRuleset {
// Convenience function to open a read-only file handle to |ruleset|.
static base::File Open(const TestRuleset& ruleset);
+ // Corrupts the |ruleset| file by truncating its tail of a certain size.
+ static void CorruptByTruncating(const TestRuleset& ruleset, size_t tail_size);
+
+ // Overrides all bytes in the [from..to) range by |fill_with|.
+ static void CorruptByFilling(const TestRuleset& ruleset,
+ size_t from,
+ size_t to,
+ uint8_t fill_with);
+
std::vector<uint8_t> contents;
base::FilePath path;
};
@@ -61,14 +70,22 @@ class TestRulesetCreator {
base::StringPiece suffix,
TestRulesetPair* test_ruleset_pair);
- void CreateRulesetWithRules(const std::vector<proto::UrlRule>& rules,
- TestRulesetPair* test_ruleset_pair);
-
// Same as above, but only creates an unindexed ruleset.
void CreateUnindexedRulesetToDisallowURLsWithPathSuffix(
base::StringPiece suffix,
TestRuleset* test_unindexed_ruleset);
+ // Similar to CreateRulesetToDisallowURLsWithPathSuffix, but the resulting
+ // ruleset consists of |num_of_suffixes| rules, each of them disallowing URLs
+ // with suffixes of the form |suffix|_k, 0 <= k < |num_of_suffixes|.
+ void CreateRulesetToDisallowURLsWithManySuffixes(
+ base::StringPiece suffix,
+ int num_of_suffixes,
+ TestRulesetPair* test_ruleset_pair);
+
+ void CreateRulesetWithRules(const std::vector<proto::UrlRule>& rules,
+ TestRulesetPair* test_ruleset_pair);
+
// Returns a unique |path| that is valid for the lifetime of this instance.
// No file at |path| will be automatically created.
void GetUniqueTemporaryPath(base::FilePath* path);

Powered by Google App Engine
This is Rietveld 408576698