Gamepad API for Android

content/browser/gamepad/gamepad_platform_data_fetcher_android.h

Index: content/browser/gamepad/gamepad_platform_data_fetcher_android.h
diff --git a/content/browser/gamepad/gamepad_platform_data_fetcher_android.h b/content/browser/gamepad/gamepad_platform_data_fetcher_android.h
new file mode 100644
index 0000000000000000000000000000000000000000..19a58a13e69aae0f8a6f2af458f1faa48d60fb41
--- /dev/null
+++ b/content/browser/gamepad/gamepad_platform_data_fetcher_android.h
@@ -0,0 +1,53 @@
+// Copyright 2014 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 CONTENT_BROWSER_GAMEPAD_GAMEPAD_DATA_FETCHER_ANDROID_H_
+#define CONTENT_BROWSER_GAMEPAD_GAMEPAD_DATA_FETCHER_ANDROID_H_
+
+#include <jni.h>
+
+#include "base/android/jni_android.h"
+#include "base/synchronization/lock.h"
+#include "content/browser/gamepad/gamepad_data_fetcher.h"
+#include "content/browser/gamepad/gamepad_standard_mappings.h"
+#include "third_party/WebKit/public/platform/WebGamepads.h"
+
+namespace content {
+
+class GamepadPlatformDataFetcherAndroid : public GamepadDataFetcher {
+ public:
+  GamepadPlatformDataFetcherAndroid();
+  virtual ~GamepadPlatformDataFetcherAndroid();
+
+  // GamepadDataFetcher implementation.
+  virtual void GetGamepadData(blink::WebGamepads* pads,
+                              bool devices_changed_hint) OVERRIDE;
+  virtual void PauseHint(bool paused);
+
+  // Called by GamepadAdapter (java).
+  void RefreshDevice(JNIEnv* env,
+                     jobject obj,
+                     int index,
+                     bool connected,
+                     jstring id,
+                     jstring mapping,
+                     long timestamp,
+                     jfloatArray axes,
+                     jfloatArray buttons);
+
+  static bool RegisterGamepadAdapter(JNIEnv* env);
+
+ private:
+  // Data to fetch into.
+  blink::WebGamepads* data_;
+
+  // The java GamepadAdapter object.
+  base::android::ScopedJavaLocalRef<jobject> j_gamepad_adapter_;

[bulach, 2014/03/05 12:34:39]

this should be ScopedJavaGlobalRef instead of Local.
the "Local" are just for, well, local variables, and they'll be potentially GCd
after the callstack is yield back to java..
"Global" is a stronger ref to the java object, and will kept it alive until it's
explicitly gone.

+
+  DISALLOW_COPY_AND_ASSIGN(GamepadPlatformDataFetcherAndroid);
+};
+
+}

[bulach, 2014/03/05 12:34:39]

nit: // namespace content

+
+#endif // CONTENT_BROWSER_GAMEPAD_GAMEPAD_DATA_FETCHER_ANDROID_H_