Avoid using ContentBrowserClient::IsIllegalOrigin in ResourceDispatcherHost.

extensions/browser/guest_view/web_view/web_view_guest.h

 // Copyright 2014 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 EXTENSIONS_BROWSER_GUEST_VIEW_WEB_VIEW_WEB_VIEW_GUEST_H_
 #define EXTENSIONS_BROWSER_GUEST_VIEW_WEB_VIEW_WEB_VIEW_GUEST_H_
 
 #include <stdint.h>
 
 #include <vector>
 
 #include "base/macros.h"
 #include "base/observer_list.h"
 #include "components/guest_view/browser/guest_view.h"
 #include "content/public/browser/javascript_dialog_manager.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "extensions/browser/guest_view/web_view/javascript_dialog_helper.h"
 #include "extensions/browser/guest_view/web_view/web_view_find_helper.h"
 #include "extensions/browser/guest_view/web_view/web_view_guest_delegate.h"
 #include "extensions/browser/guest_view/web_view/web_view_permission_helper.h"
 #include "extensions/browser/guest_view/web_view/web_view_permission_types.h"
+#include "extensions/browser/guest_view/web_view/web_view_renderer_state.h"
 #include "extensions/browser/script_executor.h"
 
 namespace blink {
 struct WebFindOptions;
 }  // namespace blink
 
 namespace content {
 struct GlobalRequestID;
+class RequestContext;
 }  // namespace content
 
 namespace extensions {
 
 class WebViewInternalFindFunction;
 
 // A WebViewGuest provides the browser-side implementation of the <webview> API
 // and manages the dispatch of <webview> extension events. WebViewGuest is
 // created on attachment. That is, when a guest WebContents is associated with
 // a particular embedder WebContents. This happens on either initial navigation
@@ skipping 273 matching lines @@
   // Creates a new guest window owned by this WebViewGuest.
   void CreateNewGuestWebViewWindow(const content::OpenURLParams& params);
 
   void NewGuestWebViewCallback(const content::OpenURLParams& params,
                                content::WebContents* guest_web_contents);
 
   bool HandleKeyboardShortcuts(const content::NativeWebKeyboardEvent& event);
 
   void ApplyAttributes(const base::DictionaryValue& params);
 
+  // Helper functions to add and remove guest processes. Invoked on the IO

[jam, 2016/08/02 17:01:46]

ditto: just move to anonymous namespace

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

Can't right now. The WebViewRendererState::AddGuest function invoked by this
class is private and the reason it works today is due to a friend relationship

+  // thread.
+  static void AddGuestHelper(
+      WebViewRendererState* renderer_state,
+      int guest_process_id,
+      int guest_routing_id,
+      const WebViewRendererState::WebViewInfo& web_view_info,
+      const content::ResourceContext* context);
+
+  static void RemoveGuestHelper(WebViewRendererState* renderer_state,
+                                int guest_process_id,
+                                int guest_routing_id,
+                                const content::ResourceContext* context);
+
   // Identifies the set of rules registries belonging to this guest.
   int rules_registry_id_;
 
   // Handles find requests and replies for the webview find API.
   WebViewFindHelper find_helper_;
 
   base::ObserverList<ScriptExecutionObserver> script_observers_;
   std::unique_ptr<ScriptExecutor> script_executor_;
 
   content::NotificationRegistrar notification_registrar_;
@@ skipping 46 matching lines @@
   // This is used to ensure pending tasks will not fire after this object is
   // destroyed.
   base::WeakPtrFactory<WebViewGuest> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WebViewGuest);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_GUEST_VIEW_WEB_VIEW_WEB_VIEW_GUEST_H_