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

Side by Side Diff: chrome/browser/extensions/event_router.cc

Issue 11359081: Lazy initialization for ProcessesEventRouter. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month 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) 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 #include "chrome/browser/extensions/event_router.h" 5 #include "chrome/browser/extensions/event_router.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
11 #include "base/message_loop.h" 11 #include "base/message_loop.h"
12 #include "base/stl_util.h" 12 #include "base/stl_util.h"
13 #include "base/values.h" 13 #include "base/values.h"
14 #include "base/version.h" 14 #include "base/version.h"
15 #include "chrome/browser/browser_process.h" 15 #include "chrome/browser/browser_process.h"
16 #include "chrome/browser/extensions/api/processes/processes_api_constants.h"
17 #include "chrome/browser/extensions/api/processes/processes_api.h"
18 #include "chrome/browser/extensions/api/runtime/runtime_api.h" 16 #include "chrome/browser/extensions/api/runtime/runtime_api.h"
19 #include "chrome/browser/extensions/api/web_request/web_request_api.h" 17 #include "chrome/browser/extensions/api/web_request/web_request_api.h"
20 #include "chrome/browser/extensions/event_names.h" 18 #include "chrome/browser/extensions/event_names.h"
21 #include "chrome/browser/extensions/extension_devtools_manager.h" 19 #include "chrome/browser/extensions/extension_devtools_manager.h"
22 #include "chrome/browser/extensions/extension_host.h" 20 #include "chrome/browser/extensions/extension_host.h"
23 #include "chrome/browser/extensions/extension_prefs.h" 21 #include "chrome/browser/extensions/extension_prefs.h"
24 #include "chrome/browser/extensions/extension_process_manager.h" 22 #include "chrome/browser/extensions/extension_process_manager.h"
25 #include "chrome/browser/extensions/extension_service.h" 23 #include "chrome/browser/extensions/extension_service.h"
26 #include "chrome/browser/extensions/extension_system.h" 24 #include "chrome/browser/extensions/extension_system.h"
27 #include "chrome/browser/extensions/lazy_background_task_queue.h" 25 #include "chrome/browser/extensions/lazy_background_task_queue.h"
(...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 } 157 }
160 158
161 void EventRouter::RemoveEventListener(const std::string& event_name, 159 void EventRouter::RemoveEventListener(const std::string& event_name,
162 content::RenderProcessHost* process, 160 content::RenderProcessHost* process,
163 const std::string& extension_id) { 161 const std::string& extension_id) {
164 EventListener listener(event_name, extension_id, process, 162 EventListener listener(event_name, extension_id, process,
165 scoped_ptr<DictionaryValue>()); 163 scoped_ptr<DictionaryValue>());
166 listeners_.RemoveListener(&listener); 164 listeners_.RemoveListener(&listener);
167 } 165 }
168 166
167 void EventRouter::RegisterObserver(Observer* observer,
168 const std::string& event_name) {
169 observers_[event_name] = observer;
170 }
171
169 void EventRouter::OnListenerAdded(const EventListener* listener) { 172 void EventRouter::OnListenerAdded(const EventListener* listener) {
170 // We don't care about lazy events being added. 173 // We don't care about lazy events being added.
171 if (!listener->process) 174 if (!listener->process)
172 return; 175 return;
173 176
177 const std::string& event_name = listener->event_name;
178 ObserverMap::iterator observer = observers_.find(event_name);
179 if (observer != observers_.end())
180 observer->second->OnListenerAdded(event_name);
181
174 if (extension_devtools_manager_.get()) 182 if (extension_devtools_manager_.get())
175 extension_devtools_manager_->AddEventListener(listener->event_name, 183 extension_devtools_manager_->AddEventListener(event_name,
176 listener->process->GetID()); 184 listener->process->GetID());
177 185
178 // We lazily tell the TaskManager to start updating when listeners to the
179 // processes.onUpdated or processes.onUpdatedWithMemory events arrive.
180 const std::string& event_name = listener->event_name;
181 if (event_name.compare(
182 extensions::processes_api_constants::kOnUpdated) == 0 ||
183 event_name.compare(
184 extensions::processes_api_constants::kOnUpdatedWithMemory) == 0)
185 extensions::ProcessesEventRouter::GetInstance()->ListenerAdded();
186
187 if (SystemInfoEventRouter::IsSystemInfoEvent(event_name)) 186 if (SystemInfoEventRouter::IsSystemInfoEvent(event_name))
188 SystemInfoEventRouter::GetInstance()->AddEventListener(event_name); 187 SystemInfoEventRouter::GetInstance()->AddEventListener(event_name);
189 } 188 }
190 189
191 void EventRouter::OnListenerRemoved(const EventListener* listener) { 190 void EventRouter::OnListenerRemoved(const EventListener* listener) {
192 // We don't care about lazy events being removed. 191 // We don't care about lazy events being removed.
193 if (!listener->process) 192 if (!listener->process)
194 return; 193 return;
195 194
196 const std::string& event_name = listener->event_name; 195 const std::string& event_name = listener->event_name;
196 ObserverMap::iterator observer = observers_.find(event_name);
197 if (observer != observers_.end())
198 observer->second->OnListenerRemoved(event_name);
199
197 if (extension_devtools_manager_.get()) 200 if (extension_devtools_manager_.get())
198 extension_devtools_manager_->RemoveEventListener( 201 extension_devtools_manager_->RemoveEventListener(
199 event_name, listener->process->GetID()); 202 event_name, listener->process->GetID());
200 203
201 // If a processes.onUpdated or processes.onUpdatedWithMemory event listener
202 // is removed (or a process with one exits), then we let the extension API
203 // know that it has one fewer listener.
204 if (event_name.compare(
205 extensions::processes_api_constants::kOnUpdated) == 0 ||
206 event_name.compare(
207 extensions::processes_api_constants::kOnUpdatedWithMemory) == 0)
208 extensions::ProcessesEventRouter::GetInstance()->ListenerRemoved();
209
210 BrowserThread::PostTask( 204 BrowserThread::PostTask(
211 BrowserThread::IO, FROM_HERE, 205 BrowserThread::IO, FROM_HERE,
212 base::Bind(&NotifyEventListenerRemovedOnIOThread, 206 base::Bind(&NotifyEventListenerRemovedOnIOThread,
213 profile_, listener->extension_id, listener->event_name)); 207 profile_, listener->extension_id, event_name));
214 208
215 if (SystemInfoEventRouter::IsSystemInfoEvent(event_name)) 209 if (SystemInfoEventRouter::IsSystemInfoEvent(event_name))
216 SystemInfoEventRouter::GetInstance()->RemoveEventListener(event_name); 210 SystemInfoEventRouter::GetInstance()->RemoveEventListener(event_name);
217 } 211 }
218 212
219 void EventRouter::AddLazyEventListener(const std::string& event_name, 213 void EventRouter::AddLazyEventListener(const std::string& event_name,
220 const std::string& extension_id) { 214 const std::string& extension_id) {
221 scoped_ptr<EventListener> listener(new EventListener( 215 scoped_ptr<EventListener> listener(new EventListener(
222 event_name, extension_id, NULL, scoped_ptr<DictionaryValue>())); 216 event_name, extension_id, NULL, scoped_ptr<DictionaryValue>()));
223 bool is_new = listeners_.AddListener(listener.Pass()); 217 bool is_new = listeners_.AddListener(listener.Pass());
(...skipping 451 matching lines...) Expand 10 before | Expand all | Expand 10 after
675 event_args(event_args.Pass()), 669 event_args(event_args.Pass()),
676 event_url(event_url), 670 event_url(event_url),
677 restrict_to_profile(restrict_to_profile), 671 restrict_to_profile(restrict_to_profile),
678 cross_incognito_args(NULL), 672 cross_incognito_args(NULL),
679 user_gesture(user_gesture), 673 user_gesture(user_gesture),
680 info(info) {} 674 info(info) {}
681 675
682 Event::~Event() {} 676 Event::~Event() {}
683 677
684 } // namespace extensions 678 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698