Revise gamepad interface

webkit/plugins/ppapi/ppapi_plugin_instance.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 "webkit/plugins/ppapi/ppapi_plugin_instance.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/memory/linked_ptr.h"
@@ skipping 202 matching lines @@
 COMPILE_ASSERT_MATCHING_ENUM(TypeCopy, PP_CURSORTYPE_COPY);
 COMPILE_ASSERT_MATCHING_ENUM(TypeNone, PP_CURSORTYPE_NONE);
 COMPILE_ASSERT_MATCHING_ENUM(TypeNotAllowed, PP_CURSORTYPE_NOTALLOWED);
 COMPILE_ASSERT_MATCHING_ENUM(TypeZoomIn, PP_CURSORTYPE_ZOOMIN);
 COMPILE_ASSERT_MATCHING_ENUM(TypeZoomOut, PP_CURSORTYPE_ZOOMOUT);
 COMPILE_ASSERT_MATCHING_ENUM(TypeGrab, PP_CURSORTYPE_GRAB);
 COMPILE_ASSERT_MATCHING_ENUM(TypeGrabbing, PP_CURSORTYPE_GRABBING);
 // Do not assert WebCursorInfo::TypeCustom == PP_CURSORTYPE_CUSTOM;
 // PP_CURSORTYPE_CUSTOM is pinned to allow new cursor types.
 
-// Ensure conversion from WebKit::WebGamepads to PP_GamepadsData_Dev is safe.
-// See also DCHECKs in SampleGamepads below.
-//
-// Temporarily disabled for 2-sided WebKit roll.
-//COMPILE_ASSERT(sizeof(WebKit::WebGamepads) == sizeof(PP_GamepadsData_Dev),
-//               size_difference);
-//COMPILE_ASSERT(sizeof(WebKit::WebGamepad) == sizeof(PP_GamepadData_Dev),
-//               size_difference);
-
 // Sets |*security_origin| to be the WebKit security origin associated with the
 // document containing the given plugin instance. On success, returns true. If
 // the instance is invalid, returns false and |*security_origin| will be
 // unchanged.
 bool SecurityOriginForInstance(PP_Instance instance_id,
                                WebKit::WebSecurityOrigin* security_origin) {
   PluginInstance* instance = HostGlobals::Get()->GetInstance(instance_id);
   if (!instance)
     return false;
 
@@ skipping 1105 matching lines @@
     NOTREACHED();
     return false;
   }
 #else
 //FIXME
   return container_->isRectTopmost(rect);
 #endif
 }
 
 void PluginInstance::SampleGamepads(PP_Instance instance,
-                                    PP_GamepadsData_Dev* data) {
-
-  // Because the WebKit objects have trivial ctors, using offsetof doesn't
-  // work. Instead use this version based on src/v8/src/globals.h. This
-  // workaround doesn't work in constant expressions as required for
-  // COMPILE_ASSERT, so DCHECK instead.
-
-#define OFFSET_OF(type, field) \
-  (reinterpret_cast<intptr_t>(&(reinterpret_cast<type*>(4)->field)) - 4)
-
-#define DCHECK_GAMEPADS_OFFSET(webkit_name, pp_name) \
-  DCHECK(OFFSET_OF(WebKit::WebGamepads, webkit_name) \
-             == OFFSET_OF(PP_GamepadsData_Dev, pp_name))
-
-#define DCHECK_GAMEPAD_OFFSET(webkit_name, pp_name) \
-  DCHECK(OFFSET_OF(WebKit::WebGamepad, webkit_name) \
-             == OFFSET_OF(PP_GamepadData_Dev, pp_name))
-
-  DCHECK_GAMEPADS_OFFSET(length, length);
-  DCHECK_GAMEPADS_OFFSET(items, items);
-  DCHECK_GAMEPAD_OFFSET(connected, connected);
-  DCHECK_GAMEPAD_OFFSET(id, id);
-  DCHECK_GAMEPAD_OFFSET(timestamp, timestamp);
-  DCHECK_GAMEPAD_OFFSET(axesLength, axes_length);
-  DCHECK_GAMEPAD_OFFSET(axes, axes);
-  DCHECK_GAMEPAD_OFFSET(buttonsLength, buttons_length);
-  DCHECK_GAMEPAD_OFFSET(buttons, buttons);
-
-#undef OFFSET_OF
-#undef DCHECK_GAMEPADS_OFFSET
-#undef DCHECK_GAMEPAD_OFFSET
-
-  delegate()->SampleGamepads(reinterpret_cast<WebKit::WebGamepads*>(data));
+                                    PP_GamepadsSampleData_Dev* data) {
+  WebKit::WebGamepads webkit_data;
+  delegate()->SampleGamepads(&webkit_data);
+  ConvertWebKitGamepadData(webkit_data, data);
 }
 
 bool PluginInstance::IsViewAccelerated() {
   if (!container_)
     return false;
 
   WebDocument document = container_->element().document();
   WebFrame* frame = document.frame();
   if (!frame)
     return false;
@@ skipping 715 matching lines @@
   screen_size_for_fullscreen_ = gfx::Size();
   WebElement element = container_->element();
   element.setAttribute(WebString::fromUTF8(kWidth), width_before_fullscreen_);
   element.setAttribute(WebString::fromUTF8(kHeight), height_before_fullscreen_);
   element.setAttribute(WebString::fromUTF8(kBorder), border_before_fullscreen_);
   element.setAttribute(WebString::fromUTF8(kStyle), style_before_fullscreen_);
 }
 
 }  // namespace ppapi
 }  // namespace webkit