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

Side by Side Diff: chrome/browser/prerender/prerender_message_filter.cc

Issue 292443004: Remove IPC_BEGIN_MESSAGE_MAP_EX macro since r270839 made all bad IPCs kill their child processes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 6 years, 7 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 | 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/prerender/prerender_message_filter.h" 5 #include "chrome/browser/prerender/prerender_message_filter.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "chrome/browser/browser_process.h" 8 #include "chrome/browser/browser_process.h"
9 #include "chrome/browser/prerender/prerender_link_manager.h" 9 #include "chrome/browser/prerender/prerender_link_manager.h"
10 #include "chrome/browser/prerender/prerender_link_manager_factory.h" 10 #include "chrome/browser/prerender/prerender_link_manager_factory.h"
(...skipping 23 matching lines...) Expand all
34 PrerenderMessageFilter::PrerenderMessageFilter(int render_process_id, 34 PrerenderMessageFilter::PrerenderMessageFilter(int render_process_id,
35 Profile* profile) 35 Profile* profile)
36 : BrowserMessageFilter(PrerenderMsgStart), 36 : BrowserMessageFilter(PrerenderMsgStart),
37 render_process_id_(render_process_id), 37 render_process_id_(render_process_id),
38 profile_(profile) { 38 profile_(profile) {
39 } 39 }
40 40
41 PrerenderMessageFilter::~PrerenderMessageFilter() { 41 PrerenderMessageFilter::~PrerenderMessageFilter() {
42 } 42 }
43 43
44 bool PrerenderMessageFilter::OnMessageReceived(const IPC::Message& message, 44 bool PrerenderMessageFilter::OnMessageReceived(const IPC::Message& message) {
45 bool* message_was_ok) {
46 bool handled = true; 45 bool handled = true;
47 IPC_BEGIN_MESSAGE_MAP_EX(PrerenderMessageFilter, message, *message_was_ok) 46 IPC_BEGIN_MESSAGE_MAP(PrerenderMessageFilter, message)
48 IPC_MESSAGE_HANDLER(PrerenderHostMsg_AddLinkRelPrerender, OnAddPrerender) 47 IPC_MESSAGE_HANDLER(PrerenderHostMsg_AddLinkRelPrerender, OnAddPrerender)
49 IPC_MESSAGE_HANDLER( 48 IPC_MESSAGE_HANDLER(
50 PrerenderHostMsg_CancelLinkRelPrerender, OnCancelPrerender) 49 PrerenderHostMsg_CancelLinkRelPrerender, OnCancelPrerender)
51 IPC_MESSAGE_HANDLER( 50 IPC_MESSAGE_HANDLER(
52 PrerenderHostMsg_AbandonLinkRelPrerender, OnAbandonPrerender) 51 PrerenderHostMsg_AbandonLinkRelPrerender, OnAbandonPrerender)
53 IPC_MESSAGE_UNHANDLED(handled = false) 52 IPC_MESSAGE_UNHANDLED(handled = false)
54 IPC_END_MESSAGE_MAP() 53 IPC_END_MESSAGE_MAP()
55 54
56 return handled; 55 return handled;
57 } 56 }
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 102 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
104 PrerenderLinkManager* prerender_link_manager = 103 PrerenderLinkManager* prerender_link_manager =
105 PrerenderLinkManagerFactory::GetForProfile(profile_); 104 PrerenderLinkManagerFactory::GetForProfile(profile_);
106 if (!prerender_link_manager) 105 if (!prerender_link_manager)
107 return; 106 return;
108 prerender_link_manager->OnAbandonPrerender(render_process_id_, prerender_id); 107 prerender_link_manager->OnAbandonPrerender(render_process_id_, prerender_id);
109 } 108 }
110 109
111 } // namespace prerender 110 } // namespace prerender
112 111
OLDNEW
« no previous file with comments | « chrome/browser/prerender/prerender_message_filter.h ('k') | chrome/browser/printing/printing_message_filter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698