gesture recognizer: Expose bounding-box for the points in a gesture-event.

content/browser/renderer_host/render_widget_host_view_win.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 "content/browser/renderer_host/render_widget_host_view_win.h"
 
 #include <algorithm>
 #include <map>
 #include <peninputpanel_i.c>
 #include <stack>
@@ skipping 382 matching lines @@
       DCHECK(false) << "Unexpected ui type. " << t;
       return WebKit::WebInputEvent::Undefined;
   }
 }
 
 class LocalGestureEvent :
     public WrappedObject<ui::GestureEvent, WebKit::WebGestureEvent> {
  public:
   LocalGestureEvent(
       HWND hwnd,
-      ui::EventType type,
+      const ui::GestureEventDetails& details,
       const gfx::Point& location,
       int flags,
       base::Time time,
-      float param_first,
-      float param_second,
       unsigned int touch_id_bitfield)
       : touch_ids_bitfield_(touch_id_bitfield),
-        type_(type) {
+        type_(details.type()) {
     // location is given in window coordinates, based on the parent window.
     // Map to the appropriate window's coordinates. For a root window the
     // coordinates won't change, because the parent shares our rect.
     POINT client_point = { location.x(), location.y()};
     MapWindowPoints(::GetParent(hwnd), hwnd, &client_point, 1);
     POINT screen_point = { location.x(), location.y()};
     MapWindowPoints(hwnd, HWND_DESKTOP, &screen_point, 1);
     data().x = client_point.x;
     data().y = client_point.y;
     data().globalX = screen_point.x;
     data().globalY = screen_point.y;
-    data().deltaX = param_first;
-    data().deltaY = param_second;
-    data().type = ConvertToWebInputEvent(type);
+    data().deltaX = details.generic_x();
+    data().deltaY = details.generic_y();
+    data().type = ConvertToWebInputEvent(type_);
+
+    // WebKit gesture events do not have bounding-boxes yet, and expect the data
+    // in deltaX/deltaY instead (and instead of bounding box, WebKit expects the
+    // radius). This is currently used only for tap events. So special case this
+    // particular case.
+    // http://crbug.com/138572
+    if (type_ == ui::ET_GESTURE_TAP) {
+      data().deltaX = details.bounding_box().width() / 2;
+      data().deltaY = details.bounding_box().height() / 2;
+    }
   }
 
   virtual int GetLowestTouchId() const OVERRIDE {
     return LowestBit(touch_ids_bitfield_);
   }
 
   ui::EventType type() {
     return type_;
   }
 
@@ skipping 844 matching lines @@
     SetToGestureMode();
   }
 }
 
 ui::GestureEvent* RenderWidgetHostViewWin::CreateGestureEvent(
     const ui::GestureEventDetails& details,
     const gfx::Point& location,
     int flags,
     base::Time time,
     unsigned int touch_id_bitfield) {
 

[sky, 2012/07/23 22:33:47]

nit: remove newline.

[sadrul, 2012/07/23 22:40:55]

Done.

-  return new LocalGestureEvent(m_hWnd, details.type(), location, flags, time,
-      details.generic_x(), details.generic_y(), touch_id_bitfield);
+  return new LocalGestureEvent(m_hWnd, details, location, flags, time,
+      touch_id_bitfield);
 }
 
 ui::TouchEvent* RenderWidgetHostViewWin::CreateTouchEvent(
     ui::EventType type,
     const gfx::Point& location,
     int touch_id,
     base::TimeDelta time_stamp) {
   return new LocalTouchEvent(type, location, touch_id, time_stamp);
 }
 
@@ skipping 1852 matching lines @@
   // receive a focus change in the context of a pointer down message, it means
   // that the pointer down message occurred on the edit field and we should
   // display the on screen keyboard
   if (!received_focus_change_after_pointer_down_ && virtual_keyboard_)
     DisplayOnScreenKeyboardIfNeeded();
   received_focus_change_after_pointer_down_ = false;
   pointer_down_context_ = false;
 }
 
 }  // namespace content