Avoid using ContentBrowserClient::IsIllegalOrigin in ResourceDispatcherHost.

content/public/browser/resource_dispatcher_host.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 CONTENT_PUBLIC_BROWSER_RESOURCE_DISPATCHER_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_RESOURCE_DISPATCHER_HOST_H_
 
 #include <stdint.h>
 
 #include <memory>
+#include <string>
 
 #include "base/callback_forward.h"
 #include "content/common/content_export.h"
 
 namespace net {
 class URLRequest;
 }
 
 namespace content {
 
@@ skipping 23 matching lines @@
   // delegate have a longer lifetime than the ResourceDispatcherHost.
   virtual void SetDelegate(ResourceDispatcherHostDelegate* delegate) = 0;
 
   // Controls whether third-party sub-content can pop-up HTTP basic auth
   // dialog boxes.
   virtual void SetAllowCrossOriginAuthPrompt(bool value) = 0;
 
   // Clears the ResourceDispatcherHostLoginDelegate associated with the request.
   virtual void ClearLoginDelegateForRequest(net::URLRequest* request) = 0;
 
+  // Specifies a scheme to be access checked. By default all schemes are
+  // allowed

[Charlie Reis, 2016/08/02 20:41:42]

nit: End with period.

Also clarify what access checked means here.  I think it means that any process
claiming to have committed a URL within the scheme must have been registered by
the methods below?

[ananta, 2016/08/02 22:28:28]

Done.

+  virtual void AddSchemeForAccessCheck(const std::string& scheme) = 0;
+
+  // The following two methods add access information for the url origin
+  // passed in. Please note that the scheme has to be registered for access
+  // check via a call to the AddSchemeForAccessCheck() method above.
+
+  // Adds |process_id| to the list of processes allowed to access the |origin|.
+  virtual void AddProcessForOrigin(const ResourceContext* context,
+                                   const std::string& origin,
+                                   int process_id) = 0;
+
+  // Removes |process_id| from the list of processes allowed to access the
+  // |origin|.
+  virtual void RemoveProcessForOrigin(const ResourceContext* context,
+                                      const std::string& origin,
+                                      int process_id) = 0;
+
  protected:
   virtual ~ResourceDispatcherHost() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RESOURCE_DISPATCHER_HOST_H_