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

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

Issue 2696493003: Introduce SubframeNavigationFilteringThrottle (Closed)
Patch Set: fix test names Created 3 years, 10 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/async_document_subresource_filter.cc
diff --git a/components/subresource_filter/content/browser/async_document_subresource_filter.cc b/components/subresource_filter/content/browser/async_document_subresource_filter.cc
index 2716f890169f5e50d6e51efc6763ca15c5e5391b..680256901dd23be9e72018ed036eadd8ce4f36eb 100644
--- a/components/subresource_filter/content/browser/async_document_subresource_filter.cc
+++ b/components/subresource_filter/content/browser/async_document_subresource_filter.cc
@@ -128,8 +128,10 @@ ActivationState AsyncDocumentSubresourceFilter::Core::Initialize(
ComputeActivationState(params.document_url, params.parent_document_origin,
params.parent_activation_state, ruleset);
- filter_.emplace(url::Origin(params.document_url), activation_state, ruleset,
- base::OnceClosure());
+ if (activation_state.activation_level != ActivationLevel::DISABLED) {
pkalinnikov 2017/02/14 12:11:06 You can remove this check, I have modified to depe
Charlie Harrison 2017/02/14 14:09:38 Done.
+ filter_.emplace(url::Origin(params.document_url), activation_state, ruleset,
+ base::OnceClosure());
+ }
return activation_state;
}

Powered by Google App Engine
This is Rietveld 408576698