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

Side by Side Diff: content/browser/renderer_host/gamepad_browser_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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "content/browser/renderer_host/gamepad_browser_message_filter.h" 5 #include "content/browser/renderer_host/gamepad_browser_message_filter.h"
6 6
7 #include "content/browser/gamepad/gamepad_service.h" 7 #include "content/browser/gamepad/gamepad_service.h"
8 #include "content/common/gamepad_messages.h" 8 #include "content/common/gamepad_messages.h"
9 9
10 namespace content { 10 namespace content {
11 11
12 GamepadBrowserMessageFilter::GamepadBrowserMessageFilter() 12 GamepadBrowserMessageFilter::GamepadBrowserMessageFilter()
13 : BrowserMessageFilter(GamepadMsgStart), 13 : BrowserMessageFilter(GamepadMsgStart),
14 is_started_(false) { 14 is_started_(false) {
15 } 15 }
16 16
17 GamepadBrowserMessageFilter::~GamepadBrowserMessageFilter() { 17 GamepadBrowserMessageFilter::~GamepadBrowserMessageFilter() {
18 DCHECK_CURRENTLY_ON(BrowserThread::IO); 18 DCHECK_CURRENTLY_ON(BrowserThread::IO);
19 if (is_started_) 19 if (is_started_)
20 GamepadService::GetInstance()->RemoveConsumer(this); 20 GamepadService::GetInstance()->RemoveConsumer(this);
21 } 21 }
22 22
23 bool GamepadBrowserMessageFilter::OnMessageReceived( 23 bool GamepadBrowserMessageFilter::OnMessageReceived(
24 const IPC::Message& message, 24 const IPC::Message& message) {
25 bool* message_was_ok) {
26 bool handled = true; 25 bool handled = true;
27 IPC_BEGIN_MESSAGE_MAP_EX(GamepadBrowserMessageFilter, 26 IPC_BEGIN_MESSAGE_MAP(GamepadBrowserMessageFilter, message)
28 message,
29 *message_was_ok)
30 IPC_MESSAGE_HANDLER(GamepadHostMsg_StartPolling, OnGamepadStartPolling) 27 IPC_MESSAGE_HANDLER(GamepadHostMsg_StartPolling, OnGamepadStartPolling)
31 IPC_MESSAGE_HANDLER(GamepadHostMsg_StopPolling, OnGamepadStopPolling) 28 IPC_MESSAGE_HANDLER(GamepadHostMsg_StopPolling, OnGamepadStopPolling)
32 IPC_MESSAGE_UNHANDLED(handled = false) 29 IPC_MESSAGE_UNHANDLED(handled = false)
33 IPC_END_MESSAGE_MAP_EX() 30 IPC_END_MESSAGE_MAP()
34 return handled; 31 return handled;
35 } 32 }
36 33
37 void GamepadBrowserMessageFilter::OnGamepadConnected( 34 void GamepadBrowserMessageFilter::OnGamepadConnected(
38 unsigned index, 35 unsigned index,
39 const blink::WebGamepad& gamepad) { 36 const blink::WebGamepad& gamepad) {
40 Send(new GamepadMsg_GamepadConnected(index, gamepad)); 37 Send(new GamepadMsg_GamepadConnected(index, gamepad));
41 } 38 }
42 39
43 void GamepadBrowserMessageFilter::OnGamepadDisconnected( 40 void GamepadBrowserMessageFilter::OnGamepadDisconnected(
(...skipping 11 matching lines...) Expand all
55 *renderer_handle = service->GetSharedMemoryHandleForProcess(PeerHandle()); 52 *renderer_handle = service->GetSharedMemoryHandleForProcess(PeerHandle());
56 } 53 }
57 54
58 void GamepadBrowserMessageFilter::OnGamepadStopPolling() { 55 void GamepadBrowserMessageFilter::OnGamepadStopPolling() {
59 CHECK(is_started_); 56 CHECK(is_started_);
60 is_started_ = false; 57 is_started_ = false;
61 GamepadService::GetInstance()->ConsumerBecameInactive(this); 58 GamepadService::GetInstance()->ConsumerBecameInactive(this);
62 } 59 }
63 60
64 } // namespace content 61 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/renderer_host/gamepad_browser_message_filter.h ('k') | content/browser/renderer_host/gpu_message_filter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698