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

Side by Side Diff: chrome/browser/automation/automation_provider.h

Issue 5978003: Make IPC::Channel::Listener:OnMessageReceived have a return value indicating ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 // This implements a browser-side endpoint for UI automation activity. 5 // This implements a browser-side endpoint for UI automation activity.
6 // The client-side endpoint is implemented by AutomationProxy. 6 // The client-side endpoint is implemented by AutomationProxy.
7 // The entire lifetime of this object should be contained within that of 7 // The entire lifetime of this object should be contained within that of
8 // the BrowserProcess, and in particular the NotificationService that's 8 // the BrowserProcess, and in particular the NotificationService that's
9 // hung off of it. 9 // hung off of it.
10 10
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
141 // Takes ownership of the container. 141 // Takes ownership of the container.
142 void AddPortContainer(ExtensionPortContainer* port); 142 void AddPortContainer(ExtensionPortContainer* port);
143 // Remove and delete the port container. 143 // Remove and delete the port container.
144 void RemovePortContainer(ExtensionPortContainer* port); 144 void RemovePortContainer(ExtensionPortContainer* port);
145 // Get the port container for the given port id. 145 // Get the port container for the given port id.
146 ExtensionPortContainer* GetPortContainer(int port_id) const; 146 ExtensionPortContainer* GetPortContainer(int port_id) const;
147 147
148 // IPC implementations 148 // IPC implementations
149 virtual bool Send(IPC::Message* msg); 149 virtual bool Send(IPC::Message* msg);
150 virtual void OnChannelConnected(int pid); 150 virtual void OnChannelConnected(int pid);
151 virtual void OnMessageReceived(const IPC::Message& msg); 151 virtual bool OnMessageReceived(const IPC::Message& msg);
152 virtual void OnChannelError(); 152 virtual void OnChannelError();
153 153
154 IPC::Message* reply_message_release() { 154 IPC::Message* reply_message_release() {
155 IPC::Message* reply_message = reply_message_; 155 IPC::Message* reply_message = reply_message_;
156 reply_message_ = NULL; 156 reply_message_ = NULL;
157 return reply_message; 157 return reply_message;
158 } 158 }
159 159
160 // Adds the extension passed in to the extension tracker, and returns 160 // Adds the extension passed in to the extension tracker, and returns
161 // the associated handle. If the tracker already contains the extension, 161 // the associated handle. If the tracker already contains the extension,
(...skipping 252 matching lines...) Expand 10 before | Expand all | Expand 10 after
414 // True iff connected to an AutomationProxy. 414 // True iff connected to an AutomationProxy.
415 bool is_connected_; 415 bool is_connected_;
416 416
417 // True iff browser finished loading initial set of tabs. 417 // True iff browser finished loading initial set of tabs.
418 bool initial_loads_complete_; 418 bool initial_loads_complete_;
419 419
420 DISALLOW_COPY_AND_ASSIGN(AutomationProvider); 420 DISALLOW_COPY_AND_ASSIGN(AutomationProvider);
421 }; 421 };
422 422
423 #endif // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_H_ 423 #endif // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_H_
OLDNEW
« no previous file with comments | « chrome/browser/appcache/appcache_dispatcher_host.cc ('k') | chrome/browser/automation/automation_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698