Pass correct length value in WebGamepads

ppapi/shared_impl/ppb_gamepad_shared.cc

 // Copyright (c) 2012 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.
 
 #include "ppapi/shared_impl/ppb_gamepad_shared.h"
 
 #include "base/basictypes.h"
 
 namespace ppapi {
 
 void ConvertWebKitGamepadData(const WebKitGamepads& webkit_data,
                               PP_GamepadsSampleData* output_data) {
-  output_data->length = webkit_data.length;
-  for (unsigned i = 0; i < webkit_data.length; ++i) {
+  output_data->length = (webkit_data.length < WebKitGamepads::kItemsLengthCap) ?
+            webkit_data.length : WebKitGamepads::kItemsLengthCap;

[arajp, 2015/05/07 12:19:54]

Added this for safety.

[bbudge, 2015/05/07 13:58:57]

std::min(webkit_data.length, WebKitGamepads::kItemsLengthCap) would be better
here.

[arajp, 2015/05/08 06:03:20]

Done.

+  for (unsigned i = 0; i < output_data->length; ++i) {
     PP_GamepadSampleData& output_pad = output_data->items[i];
     const WebKitGamepad& webkit_pad = webkit_data.items[i];
     output_pad.connected = webkit_pad.connected ? PP_TRUE : PP_FALSE;
     if (webkit_pad.connected) {
       static_assert(sizeof(output_pad.id) == sizeof(webkit_pad.id),
                     "id size does not match");
       memcpy(output_pad.id, webkit_pad.id, sizeof(output_pad.id));
       output_pad.timestamp = static_cast<double>(webkit_pad.timestamp);
       output_pad.axes_length = webkit_pad.axes_length;
       for (unsigned j = 0; j < webkit_pad.axes_length; ++j)
         output_pad.axes[j] = static_cast<float>(webkit_pad.axes[j]);
       output_pad.buttons_length = webkit_pad.buttons_length;
       for (unsigned j = 0; j < webkit_pad.buttons_length; ++j)
         output_pad.buttons[j] = static_cast<float>(webkit_pad.buttons[j].value);
     }
   }
 }
 
 }  // namespace ppapi