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

Side by Side Diff: chrome/browser/extensions/message_handler.h

Issue 10787002: Moved ExtensionMessage* into extensions namespace (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Latest master for cq Created 8 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_MESSAGE_HANDLER_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_MESSAGE_HANDLER_H_
6 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_MESSAGE_HANDLER_H_ 6 #define CHROME_BROWSER_EXTENSIONS_MESSAGE_HANDLER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "content/public/browser/render_view_host_observer.h" 10 #include "content/public/browser/render_view_host_observer.h"
11 11
12 namespace extensions {
13
12 // Filters and dispatches extension-related IPC messages that arrive from 14 // Filters and dispatches extension-related IPC messages that arrive from
13 // renderers. There is one of these objects for each RenderViewHost in Chrome. 15 // renderers. There is one of these objects for each RenderViewHost in Chrome.
14 // Contrast this with extensions::TabHelper, which is only created for 16 // Contrast this with extensions::TabHelper, which is only created for
15 // WebContents. 17 // WebContents.
16 // 18 //
17 // TODO(aa): Handling of content script messaging should be able to move to EFD 19 // TODO(aa): Handling of content script messaging should be able to move to EFD
18 // once there is an EFD for every RVHD where extension code can run. Then we 20 // once there is an EFD for every RVHD where extension code can run. Then we
19 // could eliminate this class. Right now, we don't end up with an EFD for tab 21 // could eliminate this class. Right now, we don't end up with an EFD for tab
20 // contents unless that tab contents is hosting chrome-extension:// URLs. That 22 // contents unless that tab contents is hosting chrome-extension:// URLs. That
21 // still leaves content scripts. See also: crbug.com/80307. 23 // still leaves content scripts. See also: crbug.com/80307.
22 class ExtensionMessageHandler : public content::RenderViewHostObserver { 24 class MessageHandler : public content::RenderViewHostObserver {
23 public: 25 public:
24 // |sender| is guaranteed to outlive this object. 26 // |sender| is guaranteed to outlive this object.
25 explicit ExtensionMessageHandler(content::RenderViewHost* render_view_host); 27 explicit MessageHandler(content::RenderViewHost* render_view_host);
26 virtual ~ExtensionMessageHandler(); 28 virtual ~MessageHandler();
27 29
28 // RenderViewHostObserver overrides. 30 // RenderViewHostObserver overrides.
29 virtual void RenderViewHostInitialized() OVERRIDE; 31 virtual void RenderViewHostInitialized() OVERRIDE;
30 virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE; 32 virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
31 33
32 private: 34 private:
33 // Message handlers. 35 // Message handlers.
34 void OnPostMessage(int port_id, const std::string& message); 36 void OnPostMessage(int port_id, const std::string& message);
35 37
36 DISALLOW_COPY_AND_ASSIGN(ExtensionMessageHandler); 38 DISALLOW_COPY_AND_ASSIGN(MessageHandler);
37 }; 39 };
38 40
39 #endif // CHROME_BROWSER_EXTENSIONS_EXTENSION_MESSAGE_HANDLER_H_ 41 } // namespace extensions
42
43 #endif // CHROME_BROWSER_EXTENSIONS_MESSAGE_HANDLER_H_
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_system_factory.cc ('k') | chrome/browser/extensions/message_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698