Updating Blink-side WebGamepad interface

Source/modules/gamepad/Gamepad.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
@@ skipping 10 matching lines @@
  * 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/Gamepad.h"
 
 namespace WebCore {
 
+const float ButtonPressedThreshold = 0.1f;

[scottmg, 2014/02/18 23:46:31]

As noted in the chromium-side review, I think this should happen in the data
fetcher, not at this level.

+
 DEFINE_GC_INFO(Gamepad);
 
 Gamepad::Gamepad()
     : m_index(0)
     , m_timestamp(0)
 {
     ScriptWrappable::init(this);
 }
 
 void Gamepad::axes(unsigned count, float* data)
 {
     m_axes.resize(count);
     if (count)
         std::copy(data, data + count, m_axes.begin());
 }
 
 void Gamepad::buttons(unsigned count, float* data)
 {
-    m_buttons.resize(count);
-    if (count)
-        std::copy(data, data + count, m_buttons.begin());
+    if (m_buttons.size() != count) {
+        m_buttons.resize(count);
+        for (unsigned i = 0; i < count; ++i)
+            m_buttons[i] = GamepadButton::create();
+    }
+
+    for (unsigned i = 0; i < count; ++i) {
+        m_buttons[i]->value(data[i]);
+        m_buttons[i]->pressed(data[i] >= ButtonPressedThreshold);
+    }
 }
 
 Gamepad::~Gamepad()
 {
 }
 
 void Gamepad::trace(Visitor* visitor)
 {
 }
 
 } // namespace WebCore