Index: components/policy/core/browser/url_blacklist_manager.h |
diff --git a/components/policy/core/browser/url_blacklist_manager.h b/components/policy/core/browser/url_blacklist_manager.h |
index aed642b2001086d1b87f496736aa7a26e14733a2..03e44ad19b79e213330309126b7315fe42d9f296 100644 |
--- a/components/policy/core/browser/url_blacklist_manager.h |
+++ b/components/policy/core/browser/url_blacklist_manager.h |
@@ -27,10 +27,6 @@ class ListValue; |
class SequencedTaskRunner; |
} |
-namespace net { |
-class URLRequest; |
-} |
- |
namespace user_prefs { |
class PrefRegistrySyncable; |
} |
@@ -161,14 +157,17 @@ class POLICY_EXPORT URLBlacklistManager { |
// from the IO thread. |
bool IsURLBlocked(const GURL& url) const; |
- // Returns true if |request| is blocked by the current blacklist. |
- // Only main frame and sub frame requests may be blocked; other sub resources |
- // or background downloads (e.g. extensions updates, sync, etc) are not |
- // filtered. The sync signin page is also not filtered. |
+ // Returns true if a request for |url| is blocked by the current blacklist. |
+ // |
+ // Should only be called for requests for frames (Main frames or subframes). |
+ // Other subresources or background downloads (e.g. extensions updates, sync, |
+ // etc) should not be filtered. The sync signin page will also not be |
+ // filtered. |
+ // |
// |reason| is populated with the exact reason for blocking the url if and |
// only if the return value is true otherwise it is left untouched. |
// Must be called from the IO thread. |
- bool IsRequestBlocked(const net::URLRequest& request, int* reason) const; |
+ bool ShouldBlockRequestForFrame(const GURL& url, int* reason) const; |
// Replaces the current blacklist. Must be called on the IO thread. |
// Virtual for testing. |