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

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

Issue 2838063002: [on-hold][subresource_filter] Add ActivationTracker to track all activated contents (Closed)
Patch Set: fix incognito Created 3 years, 8 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_throttle_manager.cc
diff --git a/components/subresource_filter/content/browser/content_subresource_filter_throttle_manager.cc b/components/subresource_filter/content/browser/content_subresource_filter_throttle_manager.cc
index 32b679cf7c333a3fdf17b7100dbe00feb3ea5ba8..ec2edc980e4a85b574351f7a704d7968f9f709c7 100644
--- a/components/subresource_filter/content/browser/content_subresource_filter_throttle_manager.cc
+++ b/components/subresource_filter/content/browser/content_subresource_filter_throttle_manager.cc
@@ -4,6 +4,8 @@
#include "components/subresource_filter/content/browser/content_subresource_filter_throttle_manager.h"
+#include <utility>
+
#include "base/bind.h"
#include "base/logging.h"
#include "base/memory/ptr_util.h"
@@ -105,6 +107,8 @@ void ContentSubresourceFilterThrottleManager::ReadyToCommitNavigation(
return;
}
+ if (navigation_handle->IsInMainFrame())
+ delegate_->OnPageActivated(filter->activation_state());
throttle->second->WillSendActivationToRenderer();
content::RenderFrameHost* frame_host =

Powered by Google App Engine
This is Rietveld 408576698