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

Side by Side Diff: chrome/browser/renderer_host/browser_render_process_host.cc

Issue 3117024: Clean up ExtensionMessageService. (Closed)
Patch Set: comments fixed Created 10 years, 4 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) 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 // Represents the browser side of the browser <--> renderer communication 5 // Represents the browser side of the browser <--> renderer communication
6 // channel. There will be one RenderProcessHost per renderer process. 6 // channel. There will be one RenderProcessHost per renderer process.
7 7
8 #include "chrome/browser/renderer_host/browser_render_process_host.h" 8 #include "chrome/browser/renderer_host/browser_render_process_host.h"
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 223 matching lines...) Expand 10 before | Expand all | Expand 10 after
234 while (!queued_messages_.empty()) { 234 while (!queued_messages_.empty()) {
235 delete queued_messages_.front(); 235 delete queued_messages_.front();
236 queued_messages_.pop(); 236 queued_messages_.pop();
237 } 237 }
238 238
239 // Destroy the AudioRendererHost properly. 239 // Destroy the AudioRendererHost properly.
240 if (audio_renderer_host_.get()) 240 if (audio_renderer_host_.get())
241 audio_renderer_host_->Destroy(); 241 audio_renderer_host_->Destroy();
242 242
243 ClearTransportDIBCache(); 243 ClearTransportDIBCache();
244
245 NotificationService::current()->Notify(
246 NotificationType::EXTENSION_PORT_DELETED_DEBUG,
247 Source<IPC::Message::Sender>(this),
248 NotificationService::NoDetails());
249 } 244 }
250 245
251 bool BrowserRenderProcessHost::Init(bool is_extensions_process) { 246 bool BrowserRenderProcessHost::Init(bool is_extensions_process) {
252 // calling Init() more than once does nothing, this makes it more convenient 247 // calling Init() more than once does nothing, this makes it more convenient
253 // for the view host which may not be sure in some cases 248 // for the view host which may not be sure in some cases
254 if (channel_.get()) 249 if (channel_.get())
255 return true; 250 return true;
256 251
257 // It is possible for an extension process to be reused for non-extension 252 // It is possible for an extension process to be reused for non-extension
258 // content, e.g. if an extension calls window.open. 253 // content, e.g. if an extension calls window.open.
(...skipping 812 matching lines...) Expand 10 before | Expand all | Expand 10 after
1071 IPC::InvalidPlatformFileForTransit(), 1066 IPC::InvalidPlatformFileForTransit(),
1072 std::vector<std::string>(), 1067 std::vector<std::string>(),
1073 std::string(), 1068 std::string(),
1074 false)); 1069 false));
1075 } 1070 }
1076 } 1071 }
1077 1072
1078 void BrowserRenderProcessHost::EnableAutoSpellCorrect(bool enable) { 1073 void BrowserRenderProcessHost::EnableAutoSpellCorrect(bool enable) {
1079 Send(new ViewMsg_SpellChecker_EnableAutoSpellCorrect(enable)); 1074 Send(new ViewMsg_SpellChecker_EnableAutoSpellCorrect(enable));
1080 } 1075 }
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_message_service.cc ('k') | chrome/browser/renderer_host/render_view_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698