Functionality to allow blacklist and whitelist of custom schemes

net/base/network_delegate.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_BASE_NETWORK_DELEGATE_H_
 #define NET_BASE_NETWORK_DELEGATE_H_
 
 #include <stdint.h>
 
 #include <string>
@@ skipping 37 matching lines @@
   // OnAuthRequired call. It's placed in this file to prevent url_request.h
   // from having to include network_delegate.h.
   enum AuthRequiredResponse {
     AUTH_REQUIRED_RESPONSE_NO_ACTION,
     AUTH_REQUIRED_RESPONSE_SET_AUTH,
     AUTH_REQUIRED_RESPONSE_CANCEL_AUTH,
     AUTH_REQUIRED_RESPONSE_IO_PENDING,
   };
   typedef base::Callback<void(AuthRequiredResponse)> AuthCallback;
 
+  // URLBlacklistState indicates if the URL was defined in blacklist,

[Thiemo Nagel, 2016/03/08 18:44:35]

There's no need to repeat "URLBlacklistState".

+  // in whitelist or is not present in either lists as defined in
+  // URLBlacklist and URLWhitelist policies.
+  enum URLBlacklistState {

[Thiemo Nagel, 2016/03/08 18:44:35]

Why do you need a tri-state?  Please explain in CL description.

+    URL_IN_WHITELIST,
+    URL_IN_BLACKLIST,
+    URL_NEUTRAL_STATE,
+  };
+
   virtual ~NetworkDelegate() {}
 
   // Notification interface called by the network stack. Note that these
   // functions mostly forward to the private virtuals. They also add some sanity
   // checking on parameters. See the corresponding virtuals for explanations of
   // the methods and their arguments.
   int NotifyBeforeURLRequest(URLRequest* request,
                              const CompletionCallback& callback,
                              GURL* new_url);
   void NotifyResolveProxy(const GURL& url,
@@ skipping 44 matching lines @@
   // https://crbug.com/546820
   bool AreExperimentalCookieFeaturesEnabled() const;
   // TODO(jww): Remove this once we ship strict secure cookies.
   bool AreStrictSecureCookiesEnabled() const;
 
   bool CancelURLRequestWithPolicyViolatingReferrerHeader(
       const URLRequest& request,
       const GURL& target_url,
       const GURL& referrer_url) const;
 
+  // This function is to be used to check if the url is defined in

[Thiemo Nagel, 2016/03/08 18:44:35]

Use pipes to refer to a parameter, such as |url|.

+  // blacklist or whitelist policy.
+  virtual URLBlacklistState GetURLBlacklistState(const GURL& url);

[Thiemo Nagel, 2016/03/08 18:44:35]

Please make the method const.

+
  private:
   // This is the interface for subclasses of NetworkDelegate to implement. These
   // member functions will be called by the respective public notification
   // member function, which will perform basic sanity checking.
 
   // Called before a request is sent. Allows the delegate to rewrite the URL
   // being fetched by modifying |new_url|. If set, the URL must be valid. The
   // reference fragment from the original URL is not automatically appended to
   // |new_url|; callers are responsible for copying the reference fragment if
   // desired.
@@ skipping 178 matching lines @@
   // header is stripped from the request.
   virtual bool OnCancelURLRequestWithPolicyViolatingReferrerHeader(
       const URLRequest& request,
       const GURL& target_url,
       const GURL& referrer_url) const = 0;
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_NETWORK_DELEGATE_H_