Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6033)

Unified Diff: chrome/browser/extensions/extension_host.h

Issue 6794035: Move dispatching and sending of the last extension specific messages out of TabContents and Rende... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_host.h
===================================================================
--- chrome/browser/extensions/extension_host.h (revision 80315)
+++ chrome/browser/extensions/extension_host.h (working copy)
@@ -29,6 +29,7 @@
class DesktopNotificationHandler;
class DevToolsHandler;
class Extension;
+class ExtensionMessageHandler;
class FileSelectHelper;
class RenderProcessHost;
class RenderWidgetHostView;
@@ -130,8 +131,6 @@
// RenderViewHostDelegate implementation.
virtual RenderViewHostDelegate::View* GetViewDelegate();
virtual WebPreferences GetWebkitPrefs();
- virtual void ProcessWebUIMessage(
- const ExtensionHostMsg_DomMessage_Params& params);
virtual void RunJavaScriptMessage(const std::wstring& message,
const std::wstring& default_prompt,
const GURL& frame_url,
@@ -232,13 +231,16 @@
// Message handlers.
void OnRunFileChooser(const ViewHostMsg_RunFileChooser_Params& params);
- void OnPostMessage(int port_id, const std::string& message);
// Handles keyboard events that were not handled by HandleKeyboardEvent().
// Platform specific implementation may override this method to handle the
// event in platform specific way.
virtual void UnhandledKeyboardEvent(const NativeWebKeyboardEvent& event) {}
+ // Updates extension_function_dispatcher_. Call this instead of modifying it
+ // directly.
+ void SetExtensionFunctionDispatcher(ExtensionFunctionDispatcher* efd);
+
// Returns true if we're hosting a background page.
// This isn't valid until CreateRenderView is called.
bool is_background_page() const { return !view(); }
@@ -296,6 +298,9 @@
// Filters dev tools IPCs.
scoped_ptr<DevToolsHandler> dev_tools_handler_;
+ // Handles extension IPCs.
+ scoped_ptr<ExtensionMessageHandler> extension_message_handler_;
+
// The time that the last javascript message was dismissed.
base::TimeTicks last_javascript_message_dismissal_;

Powered by Google App Engine
This is Rietveld 408576698