[BlackBerry] Padding adjustment for determining the relative distance for a point...
authormifenton@rim.com <mifenton@rim.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 24 May 2012 19:29:19 +0000 (19:29 +0000)
committermifenton@rim.com <mifenton@rim.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 24 May 2012 19:29:19 +0000 (19:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=87404

Reviewed by Rob Buis.

Fix padding calculation for handle adjustment.

Reviewed Internally by Mike Lattanzio.

* WebKitSupport/SelectionHandler.cpp:
(BlackBerry::WebKit::directionOfPointRelativeToRect):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@118403 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/blackberry/ChangeLog
Source/WebKit/blackberry/WebKitSupport/SelectionHandler.cpp

index 7aa18d0..e6034b7 100644 (file)
@@ -1,5 +1,19 @@
 2012-05-24  Mike Fenton  <mifenton@rim.com>
 
+        [BlackBerry] Padding adjustment for determining the relative distance for a point was backwards.
+        https://bugs.webkit.org/show_bug.cgi?id=87404
+
+        Reviewed by Rob Buis.
+
+        Fix padding calculation for handle adjustment.
+
+        Reviewed Internally by Mike Lattanzio.
+
+        * WebKitSupport/SelectionHandler.cpp:
+        (BlackBerry::WebKit::directionOfPointRelativeToRect):
+
+2012-05-24  Mike Fenton  <mifenton@rim.com>
+
         [BlackBerry] Remove unused API function touchEventCancelAndClearFocusedNode.
         https://bugs.webkit.org/show_bug.cgi?id=87379
 
index 351f527..da43f2c 100644 (file)
@@ -177,7 +177,7 @@ static unsigned short directionOfPointRelativeToRect(const WebCore::IntPoint& po
 
     // Do height movement check first but add padding. We may be off on both x & y axis and only
     // want to move in one direction at a time.
-    if (point.y() + (useTopPadding ? verticalPadding : 0) < rect.y())
+    if (point.y() - (useTopPadding ? verticalPadding : 0) < rect.y())
         return KEYCODE_UP;
     if (point.y() > rect.maxY() + (useBottomPadding ? verticalPadding : 0))
         return KEYCODE_DOWN;