Implement NavigationThrottle::BLOCK_REQUEST_AND_COLLAPSE.

components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h

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 e0c8aefc902946f5b5d09c20c893f2ee1f255d8c..750b976d67736e8500c283fd7ec669658914c8fd 100644
--- a/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h
+++ b/components/subresource_filter/content/browser/subframe_navigation_filtering_throttle.h
@@ -40,8 +40,9 @@ class SubframeNavigationFilteringThrottle : public content::NavigationThrottle {
   const char* GetNameForLogging() override;
 
  private:
-  content::NavigationThrottle::ThrottleCheckResult DeferToCalculateLoadPolicy();
-  void OnCalculatedLoadPolicy(LoadPolicy policy);
+  content::NavigationThrottle::ThrottleCheckResult DeferToCalculateLoadPolicy(
+      bool is_redirect);

[dcheng, 2017/05/10 06:59:46]

It would be nice if this were an enum instead of a bool (it's not easy to tell
from the function name what the bool means, so using bools ends up requiring
extra comments anyway)

[engedy, 2017/05/15 13:35:51]

Good point, done.

+  void OnCalculatedLoadPolicy(bool was_deferring_redirect, LoadPolicy policy);

[dcheng, 2017/05/10 06:59:46]

Ditto

[engedy, 2017/05/15 13:35:51]

Done.

 
   // Must outlive this class.
   AsyncDocumentSubresourceFilter* parent_frame_filter_;