Remove LOAD_SUB_FRAME load flag.

components/policy/core/browser/url_blacklist_manager.h

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..872c1283db6bb26ad7b29f7957c8503e152feb6d 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,18 @@ 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.
+  // |is_request_for_frame| must be true if the request is for a main frame
+  // or sub frame.
+  // Only requests fo frames may be blocked; other sub resources or background

[davidben, 2015/05/12 00:49:47]

fo -> for

[davidben, 2015/05/12 00:49:47]

Why bother calling this function when is_request_for_frame is false anyway? You
already know it's not going to do anything.

[mmenke, 2015/05/12 16:11:40]

Was just thinking from the perspective of keeping logic out of
ChromeNetworkDelegate, given all the random cruft it has, and the fact that it's
not unit tests.  Does seem a bit silly, though.  I've gone ahead and removed the
argument.

[mmenke, 2015/05/12 16:11:40]

Done.

+  // downloads (e.g. extensions updates, sync, etc) are not filtered. The sync
+  // signin page is also not 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 IsRequestBlocked(const GURL& url,
+                        bool is_request_for_frame,
+                        int* reason) const;
 
   // Replaces the current blacklist. Must be called on the IO thread.
   // Virtual for testing.