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

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: add some more comments Created 6 years, 9 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(IPC::Sender* sender)
13 : BrowserMessageFilter(GamepadMsgStart), 13 : BrowserMessageFilter(GamepadMsgStart),
14 sender_(sender),
14 is_started_(false) { 15 is_started_(false) {
15 } 16 }
16 17
17 GamepadBrowserMessageFilter::~GamepadBrowserMessageFilter() { 18 GamepadBrowserMessageFilter::~GamepadBrowserMessageFilter() {
18 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 19 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
19 if (is_started_) 20 if (is_started_)
20 GamepadService::GetInstance()->RemoveConsumer(); 21 GamepadService::GetInstance()->RemoveConsumer(this);
21 } 22 }
22 23
23 bool GamepadBrowserMessageFilter::OnMessageReceived( 24 bool GamepadBrowserMessageFilter::OnMessageReceived(
24 const IPC::Message& message, 25 const IPC::Message& message,
25 bool* message_was_ok) { 26 bool* message_was_ok) {
26 bool handled = true; 27 bool handled = true;
27 IPC_BEGIN_MESSAGE_MAP_EX(GamepadBrowserMessageFilter, 28 IPC_BEGIN_MESSAGE_MAP_EX(GamepadBrowserMessageFilter,
28 message, 29 message,
29 *message_was_ok) 30 *message_was_ok)
30 IPC_MESSAGE_HANDLER(GamepadHostMsg_StartPolling, OnGamepadStartPolling) 31 IPC_MESSAGE_HANDLER(GamepadHostMsg_StartPolling, OnGamepadStartPolling)
31 IPC_MESSAGE_HANDLER(GamepadHostMsg_StopPolling, OnGamepadStopPolling) 32 IPC_MESSAGE_HANDLER(GamepadHostMsg_StopPolling, OnGamepadStopPolling)
32 IPC_MESSAGE_UNHANDLED(handled = false) 33 IPC_MESSAGE_UNHANDLED(handled = false)
33 IPC_END_MESSAGE_MAP_EX() 34 IPC_END_MESSAGE_MAP_EX()
34 return handled; 35 return handled;
35 } 36 }
36 37
38 void GamepadBrowserMessageFilter::OnGamepadConnected(
39 unsigned index,
40 const blink::WebGamepad& web_gamepad) {
41 sender_->Send(new GamepadMsg_GamepadConnected(
42 GamepadConnectionEventMessageParams(index, web_gamepad)));
43 }
44
45 void GamepadBrowserMessageFilter::OnGamepadDisconnected(
46 unsigned index,
47 const blink::WebGamepad& web_gamepad) {
48 sender_->Send(new GamepadMsg_GamepadDisconnected(
49 GamepadConnectionEventMessageParams(index, web_gamepad)));
50 }
51
37 void GamepadBrowserMessageFilter::OnGamepadStartPolling( 52 void GamepadBrowserMessageFilter::OnGamepadStartPolling(
38 base::SharedMemoryHandle* renderer_handle) { 53 base::SharedMemoryHandle* renderer_handle) {
39 GamepadService* service = GamepadService::GetInstance(); 54 GamepadService* service = GamepadService::GetInstance();
40 if (!is_started_) { 55 CHECK(!is_started_);
41 is_started_ = true; 56 is_started_ = true;
42 service->AddConsumer(); 57 service->AddConsumer(this);
43 *renderer_handle = service->GetSharedMemoryHandleForProcess(PeerHandle()); 58 *renderer_handle = service->GetSharedMemoryHandleForProcess(PeerHandle());
44 } else {
45 // Currently we only expect the renderer to tell us once to start.
46 NOTREACHED();
47 }
48 } 59 }
49 60
50 void GamepadBrowserMessageFilter::OnGamepadStopPolling() { 61 void GamepadBrowserMessageFilter::OnGamepadStopPolling() {
51 // TODO(scottmg): Probably get rid of this message. We can't trust it will 62 CHECK(is_started_);
52 // arrive anyway if the renderer crashes, etc. 63 is_started_ = false;
53 if (is_started_) { 64 GamepadService::GetInstance()->RemoveConsumer(this);
54 is_started_ = false;
55 GamepadService::GetInstance()->RemoveConsumer();
56 } else {
57 NOTREACHED();
58 }
59 } 65 }
60 66
61 } // namespace content 67 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698