Make PepperWebPlugin not use RenderViews.

chrome/browser/extensions/extension_function_dispatcher.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 CHROME_BROWSER_EXTENSIONS_EXTENSION_FUNCTION_DISPATCHER_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_FUNCTION_DISPATCHER_H_
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/memory/weak_ptr.h"
 #include "extensions/browser/extension_function.h"
 #include "ipc/ipc_sender.h"
 #include "url/gurl.h"
 
 class ChromeRenderMessageFilter;
 struct ExtensionHostMsg_Request_Params;
 
 namespace content {
 class BrowserContext;
+class RenderFrameHost;
 class RenderViewHost;
 class WebContents;
 }
 
 namespace extensions {
 class Extension;
 class ExtensionAPI;
 class InfoMap;
 class ProcessMap;
 class WindowController;
@@ skipping 65 matching lines @@
   // - This object outlives any RenderViewHost's passed to created
   //   ExtensionFunctions.
   ExtensionFunctionDispatcher(content::BrowserContext* browser_context,
                               Delegate* delegate);
 
   ~ExtensionFunctionDispatcher();
 
   Delegate* delegate() { return delegate_; }
 
   // Message handlers.
   // The response is sent to the corresponding render view in an

[Yoyo Zhou, 2013/12/20 00:28:37]

Does this comment need updating?

[jam, 2013/12/20 17:00:48]

Done.

   // ExtensionMsg_Response message.
   void Dispatch(const ExtensionHostMsg_Request_Params& params,
                 content::RenderViewHost* render_view_host);

[yzshen1, 2013/12/20 01:52:49]

Now the two Dispatch.*() methods don't use the same Render.*Host parameter. I
guess it is because you don't want to touch all other extension-related code in
this same CL?

If there is a plan to also change that part, maybe we could add a TODO.

[jam, 2013/12/20 17:00:48]

exactly
added

   // |callback| is called when the function execution completes.
   void DispatchWithCallback(
       const ExtensionHostMsg_Request_Params& params,
-      content::RenderViewHost* render_view_host,
+      content::RenderFrameHost* render_frame_host,
       const ExtensionFunction::ResponseCallback& callback);
 
   // Called when an ExtensionFunction is done executing, after it has sent
   // a response (if any) to the extension.
   void OnExtensionFunctionCompleted(const extensions::Extension* extension);
 
   // The BrowserContext that this dispatcher is associated with.
   content::BrowserContext* browser_context() { return browser_context_; }
 
  private:
@@ skipping 24 matching lines @@
       const extensions::ProcessMap& process_map,
       extensions::ExtensionAPI* api,
       void* profile,
       const ExtensionFunction::ResponseCallback& callback);
 
   // Helper to run the response callback with an access denied error. Can be
   // called on any thread.
   static void SendAccessDenied(
       const ExtensionFunction::ResponseCallback& callback);
 
+  void DispatchWithCallbackInternal(
+      const ExtensionHostMsg_Request_Params& params,
+      content::RenderViewHost* render_view_host,
+      content::RenderFrameHost* render_frame_host,
+      const ExtensionFunction::ResponseCallback& callback);
+
   content::BrowserContext* browser_context_;
 
   Delegate* delegate_;
 
   // This map doesn't own either the keys or the values. When a RenderViewHost
   // instance goes away, the corresponding entry in this map (if exists) will be
   // removed.
   typedef std::map<content::RenderViewHost*, UIThreadResponseCallbackWrapper*>
       UIThreadResponseCallbackWrapperMap;
   UIThreadResponseCallbackWrapperMap ui_thread_response_callback_wrappers_;
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_FUNCTION_DISPATCHER_H_