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

Unified Diff: components/exo/gamepad.h

Issue 2076013002: exo: Implement wayland gamepad support (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@serv
Patch Set: fixed polling thread. Created 4 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: components/exo/gamepad.h
diff --git a/components/exo/gamepad.h b/components/exo/gamepad.h
new file mode 100644
index 0000000000000000000000000000000000000000..c462c167290b7c9f707ade3ef194098b663fcf9a
--- /dev/null
+++ b/components/exo/gamepad.h
@@ -0,0 +1,76 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_EXO_GAMEPAD_H_
+#define COMPONENTS_EXO_GAMEPAD_H_
+
+#include <memory>
+
+#include "base/macros.h"
+#include "base/sequenced_task_runner.h"
+#include "base/threading/thread.h"
+#include "base/threading/thread_task_runner_handle.h"
+#include "device/gamepad/gamepad_data_fetcher.h"
+
+namespace exo {
+class GamepadDelegate;
+
+// This class represents a gamepad, it implements a background thread
reveman 2016/06/29 21:55:44 s/a gamepad/one or more gamepads/ as discussed
denniskempin 2016/06/30 00:29:21 Done.
+// for polling gamepad devices and notifies the GamepadDelegate of any
+// changes.
+class Gamepad {
+ public:
+ // This class will post tasks to invoke the delegate on the thread runner
+ // which is associated with the thread that is creating this instance.
+ Gamepad(size_t id, GamepadDelegate* delegate);
reveman 2016/06/29 21:55:45 remove id as discussed
denniskempin 2016/06/30 00:29:22 Done.
+
+ // Allows test cases to specify a GamepadDataFetcher to provide mock gamepad
+ // data.
+ Gamepad(size_t id,
reveman 2016/06/29 21:55:45 remove id as discussed
denniskempin 2016/06/30 00:29:22 Done.
+ GamepadDelegate* delegate,
+ std::unique_ptr<device::GamepadDataFetcher> fetcher);
+ virtual ~Gamepad();
+
+ private:
+ // Schedules a poll on the polling thread. Can be called from any thread.
+ void ScheduleOnPoll();
+
+ // Polls devices for new data and posts delegate updates.
+ // Can only be called on the polling thread.
+ void OnPoll();
+
+ // Posts updates of gamepad data to delegate.
+ // Can only be called on the polling thread.
+ void PostGamepadChanges(const blink::WebGamepads new_state);
+
+ // Initializes the underlying gamepad data fetcher.
+ // Can only be called on the polling thread.
+ void InitializePollingThread();
+
+ // Gamepad id
+ size_t id_;
reveman 2016/06/29 21:55:45 remove as discussed
denniskempin 2016/06/30 00:29:21 Done.
+
+ // The delegate instance that all events are dispatched to.
+ GamepadDelegate* const delegate_;
+
+ // Implements the logic to fetch gamepad information from connected devices.
+ std::unique_ptr<device::GamepadDataFetcher> fetcher_;
+
+ // The thread on which the polling is executed.
+ std::unique_ptr<base::Thread> polling_thread_;
reveman 2016/06/29 21:55:45 fyi, one thread per gamepad interface instances is
denniskempin 2016/06/30 00:29:21 I agree. We will need some kind of shared service
+
+ // The current state of gamepad.
+ blink::WebGamepads state_;
+
+ // Reference to the task runner of the thread that created this instance.
+ scoped_refptr<base::SingleThreadTaskRunner> origin_task_runner_;
+
+ base::ThreadChecker thread_checker_;
+
+ DISALLOW_COPY_AND_ASSIGN(Gamepad);
+};
+
+} // namespace exo
+
+#endif // COMPONENTS_EXO_GAMEPAD_H_
« no previous file with comments | « components/exo/DEPS ('k') | components/exo/gamepad.cc » ('j') | components/exo/gamepad.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698