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

Side by Side Diff: content/browser/renderer_host/gamepad_browser_message_filter.cc

Issue 195873019: Gamepad API: add support for connection events (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: and a missing override 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
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(); 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) { 25 bool* message_was_ok) {
26 bool handled = true; 26 bool handled = true;
27 IPC_BEGIN_MESSAGE_MAP_EX(GamepadBrowserMessageFilter, 27 IPC_BEGIN_MESSAGE_MAP_EX(GamepadBrowserMessageFilter,
28 message, 28 message,
29 *message_was_ok) 29 *message_was_ok)
30 IPC_MESSAGE_HANDLER(GamepadHostMsg_StartPolling, OnGamepadStartPolling) 30 IPC_MESSAGE_HANDLER(GamepadHostMsg_StartPolling, OnGamepadStartPolling)
31 IPC_MESSAGE_HANDLER(GamepadHostMsg_StopPolling, OnGamepadStopPolling) 31 IPC_MESSAGE_HANDLER(GamepadHostMsg_StopPolling, OnGamepadStopPolling)
32 IPC_MESSAGE_UNHANDLED(handled = false) 32 IPC_MESSAGE_UNHANDLED(handled = false)
33 IPC_END_MESSAGE_MAP_EX() 33 IPC_END_MESSAGE_MAP_EX()
34 return handled; 34 return handled;
35 } 35 }
36 36
37 void GamepadBrowserMessageFilter::OnGamepadConnected(
38 unsigned index,
39 const blink::WebGamepad& gamepad) {
40 Send(new GamepadMsg_GamepadConnected(index, gamepad));
41 }
42
43 void GamepadBrowserMessageFilter::OnGamepadDisconnected(
44 unsigned index,
45 const blink::WebGamepad& gamepad) {
46 Send(new GamepadMsg_GamepadDisconnected(index, gamepad));
47 }
48
37 void GamepadBrowserMessageFilter::OnGamepadStartPolling( 49 void GamepadBrowserMessageFilter::OnGamepadStartPolling(
38 base::SharedMemoryHandle* renderer_handle) { 50 base::SharedMemoryHandle* renderer_handle) {
39 GamepadService* service = GamepadService::GetInstance(); 51 GamepadService* service = GamepadService::GetInstance();
40 if (!is_started_) { 52 CHECK(!is_started_);
41 is_started_ = true; 53 is_started_ = true;
42 service->AddConsumer(); 54 service->ConsumerBecameActive(this);
43 *renderer_handle = service->GetSharedMemoryHandleForProcess(PeerHandle()); 55 *renderer_handle = service->GetSharedMemoryHandleForProcess(PeerHandle());
44 } else {
45 // Currently we only expect the renderer to tell us once to start.
46 NOTREACHED();
47 }
48 } 56 }
49 57
50 void GamepadBrowserMessageFilter::OnGamepadStopPolling() { 58 void GamepadBrowserMessageFilter::OnGamepadStopPolling() {
51 // TODO(scottmg): Probably get rid of this message. We can't trust it will 59 CHECK(is_started_);
52 // arrive anyway if the renderer crashes, etc. 60 is_started_ = false;
53 if (is_started_) { 61 GamepadService::GetInstance()->ConsumerBecameInactive(this);
54 is_started_ = false;
55 GamepadService::GetInstance()->RemoveConsumer();
56 } else {
57 NOTREACHED();
58 }
59 } 62 }
60 63
61 } // namespace content 64 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698