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

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

Issue 2288023003: Cache is_third_party requests in a one-entry cache. (Closed)
Patch Set: Address more comments from engedy@ Created 4 years, 2 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/indexed_ruleset.h
diff --git a/components/subresource_filter/core/common/indexed_ruleset.h b/components/subresource_filter/core/common/indexed_ruleset.h
index fd65b5cf3599300be2c73135d2b0ff5d75bfb827..947cb332c344621577a35c950e5cf355ff061b8b 100644
--- a/components/subresource_filter/core/common/indexed_ruleset.h
+++ b/components/subresource_filter/core/common/indexed_ruleset.h
@@ -21,6 +21,8 @@
namespace subresource_filter {
+class FirstPartyOrigin;
+
// The integer type used to represent N-grams.
using NGram = uint64_t;
// The hasher used for hashing N-grams.
@@ -127,7 +129,7 @@ class IndexedRulesetMatcher {
// |document_origin| is not allowed to proceed. Always returns false if the
// |url| is not valid or |element_type| == ELEMENT_TYPE_UNSPECIFIED.
bool ShouldDisallowResourceLoad(const GURL& url,
- const url::Origin& document_origin,
+ const FirstPartyOrigin& first_party,
proto::ElementType element_type) const;
private:

Powered by Google App Engine
This is Rietveld 408576698