Gamepad API: add support for gamepadconnected and gamepaddisconnected events

Source/modules/gamepad/NavigatorGamepad.cpp

 /*
  * Copyright (C) 2011, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
  *
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  *
  * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
  * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
  * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH
  * DAMAGE.
  */
 
 #include "config.h"
 #include "modules/gamepad/NavigatorGamepad.h"
 
+#include "core/frame/DOMWindow.h"
+#include "core/frame/LocalFrame.h"
 #include "core/frame/Navigator.h"
+#include "modules/gamepad/GamepadEvent.h"
 #include "modules/gamepad/GamepadList.h"
 #include "modules/gamepad/WebKitGamepadList.h"
 #include "public/platform/Platform.h"
 #include "wtf/PassOwnPtr.h"
 
 namespace WebCore {
 
 template<typename T>
 static void sampleGamepad(unsigned index, T& gamepad, const blink::WebGamepad& webGamepad)
 {
@@ skipping 20 matching lines @@
             if (!gamepad)
                 gamepad = GamepadType::create();
             sampleGamepad(i, *gamepad, webGamepad);
             into->set(i, gamepad);
         } else {
             into->set(i, nullptr);
         }
     }
 }
 
-NavigatorGamepad::NavigatorGamepad()
+NavigatorGamepad::NavigatorGamepad(LocalFrame* frame)
+    : DOMWindowProperty(frame)
 {
+    blink::Platform::current()->setGamepadListener(this);
 }
 
 NavigatorGamepad::~NavigatorGamepad()
 {
+    blink::Platform::current()->setGamepadListener(0);
 }
 
 const char* NavigatorGamepad::supplementName()
 {
     return "NavigatorGamepad";
 }
 
 NavigatorGamepad& NavigatorGamepad::from(Navigator& navigator)
 {
     NavigatorGamepad* supplement = static_cast<NavigatorGamepad*>(Supplement<Navigator>::from(navigator, supplementName()));
     if (!supplement) {
-        supplement = new NavigatorGamepad();
+        supplement = new NavigatorGamepad(navigator.frame());
         provideTo(navigator, supplementName(), adoptPtr(supplement));
     }
     return *supplement;
 }
 
 WebKitGamepadList* NavigatorGamepad::webkitGetGamepads(Navigator& navigator)
 {
     return NavigatorGamepad::from(navigator).webkitGamepads();
 }
 
@@ skipping 11 matching lines @@
 }
 
 GamepadList* NavigatorGamepad::gamepads()
 {
     if (!m_gamepads)
         m_gamepads = GamepadList::create();
     sampleGamepads<Gamepad>(m_gamepads.get());
     return m_gamepads.get();
 }
 
+void NavigatorGamepad::onGamepadConnected(unsigned index, const blink::WebGamepad& webGamepad)
+{
+    if (index >= blink::WebGamepads::itemsLengthCap || !webGamepad.connected)
+        return;
+
+    if (!m_gamepads)
+        m_gamepads = GamepadList::create();
+
+    RefPtrWillBeRawPtr<Gamepad> gamepad = m_gamepads->item(index);
+    if (!gamepad)
+        gamepad = Gamepad::create();
+    sampleGamepad(index, *gamepad, webGamepad);
+    m_gamepads->set(index, gamepad);
+
+    RefPtr<GamepadEvent> event = GamepadEvent::create(EventTypeNames::gamepadconnected, false, true, gamepad.get());
+    frame()->domWindow()->dispatchEvent(event);

[Inactive, 2014/03/14 19:46:08]

We probably need a null check for the frame

[kbalazs, 2014/03/14 23:51:27]

Yep, null check added for frame() and frame()->domWindow().

+}
+
+void NavigatorGamepad::onGamepadDisconnected(unsigned index, const blink::WebGamepad& webGamepad)
+{
+    if (index >= blink::WebGamepads::itemsLengthCap)
+        return;
+
+    if (!m_gamepads)
+        m_gamepads = GamepadList::create();
+
+    RefPtrWillBeRawPtr<Gamepad> gamepad = m_gamepads->item(index);
+    if (!gamepad || gamepad->id().characters16() != webGamepad.id)

[Inactive, 2014/03/14 19:46:08]

Comparison pointers instead of comparing strings? Doesn't look right.

[kbalazs, 2014/03/14 23:51:27]

Oops, I didn't really thought it over. I don't think this condition is needed at
all so I removed.

+        gamepad = Gamepad::create();
+    sampleGamepad(index, *gamepad, webGamepad);
+
+    RefPtr<GamepadEvent> event = GamepadEvent::create(EventTypeNames::gamepaddisconnected, false, true, gamepad);
+    frame()->domWindow()->dispatchEvent(event);

[Inactive, 2014/03/14 19:46:08]

We probably need a null check for the frame

[kbalazs, 2014/03/14 23:51:27]

Done.

+}
+
 } // namespace WebCore