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

Unified Diff: components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h

Issue 2632633006: Implement NavigationThrottle::BLOCK_REQUEST_AND_COLLAPSE. (Closed)
Patch Set: Fix navigation transition type. Created 3 years, 7 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/subframe_navigation_filtering_throttle.h
diff --git a/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h b/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h
index 1757b576132fff427bb901aba026e0a2e77a2904..e4dbfe12c846961fd56e1076418ecdf9d2c57c96 100644
--- a/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h
+++ b/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h
@@ -41,8 +41,11 @@ class SubframeNavigationFilteringThrottle : public content::NavigationThrottle {
const char* GetNameForLogging() override;
private:
- content::NavigationThrottle::ThrottleCheckResult DeferToCalculateLoadPolicy();
- void OnCalculatedLoadPolicy(LoadPolicy policy);
+ enum class ThrottlingStage { WillStartRequest, WillRedirectRequest };
+
+ content::NavigationThrottle::ThrottleCheckResult DeferToCalculateLoadPolicy(
+ ThrottlingStage stage);
+ void OnCalculatedLoadPolicy(ThrottlingStage stage, LoadPolicy policy);
// Must outlive this class.
AsyncDocumentSubresourceFilter* parent_frame_filter_;

Powered by Google App Engine
This is Rietveld 408576698