Changeset 116983 in webkit


Ignore:
Timestamp:
May 14, 2012 12:24:16 PM (12 years ago)
Author:
commit-queue@webkit.org
Message:

[chromium] Compute the best target node on a GestureTap event
​https://bugs.webkit.org/show_bug.cgi?id=85101

Patch by Terry Anderson <​tdanderson@chromium.org> on 2012-05-14
Reviewed by Adam Barth.

Source/WebCore:

Using tests in the touchadjustment/ directory.

  • WebCore.gypi:

Include TouchAdjustment.* files.

  • page/TouchAdjustment.cpp:

Included math.h because of an 'Undeclared identifier: INFINITY' when
this patch was landed before.

Source/WebKit/chromium:

  • features.gypi:

Enable the touch adjustment code by default.

  • src/WebInputEventConversion.cpp:

(WebKit::PlatformGestureEventBuilder::PlatformGestureEventBuilder):
In the case of a GestureTap, deltaX represents the horizontal padding of the touch
area and deltaY represents the vertical padding of the touch area.

LayoutTests:

  • platform/chromium/test_expectations.txt:

The flaky tests in touchadjustment/ have been addressed in WK86218.

Location:
trunk
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r116980 r116983  
     12012-05-14  Terry Anderson  <tdanderson@chromium.org>
     2
     3        [chromium] Compute the best target node on a GestureTap event
     4        https://bugs.webkit.org/show_bug.cgi?id=85101
     5
     6        Reviewed by Adam Barth.
     7
     8        * platform/chromium/test_expectations.txt:
     9        The flaky tests in touchadjustment/ have been addressed in WK86218.
     10
    1112012-05-14  Jian Li  <jianli@chromium.org>
    212

  • trunk/LayoutTests/platform/chromium/test_expectations.txt

    r116980 r116983  
    143143// Battery Status API is not supported yet in the chromium port.
    144144BUGWK62698 SKIP : batterystatus = PASS FAIL
    145 
    146 // Touch Adjustment is not supported yet in the chromium port.
    147 BUGWK78801 SKIP : touchadjustment/ = FAIL
    148145
    149146// Network Information API is not supported yet in the chromium port.

  • trunk/Source/WebCore/ChangeLog

    r116979 r116983  
     12012-05-14  Terry Anderson  <tdanderson@chromium.org>
     2
     3        [chromium] Compute the best target node on a GestureTap event
     4        https://bugs.webkit.org/show_bug.cgi?id=85101
     5
     6        Reviewed by Adam Barth.
     7
     8        Using tests in the touchadjustment/ directory.
     9
     10        * WebCore.gypi:
     11        Include TouchAdjustment.* files.
     12        * page/TouchAdjustment.cpp:
     13        Included math.h because of an 'Undeclared identifier: INFINITY' when
     14        this patch was landed before.
     15
    1162012-05-14  Adrienne Walker  <enne@google.com>
    217

  • trunk/Source/WebCore/WebCore.gypi

    r116974 r116983  
    30573057            'page/SuspendableTimer.cpp',
    30583058            'page/SuspendableTimer.h',
     3059            'page/TouchAdjustment.cpp',
     3060            'page/TouchAdjustment.h',
    30593061            'page/UserContentURLPattern.cpp',
    30603062            'page/WebKitAnimation.cpp',

  • trunk/Source/WebCore/page/TouchAdjustment.cpp

    r116650 r116983  
    3535#include "RenderObject.h"
    3636#include "RenderStyle.h"
     37#include <math.h>
    3738
    3839namespace WebCore {

  • trunk/Source/WebKit/chromium/ChangeLog

    r116979 r116983  
     12012-05-14  Terry Anderson  <tdanderson@chromium.org>
     2
     3        [chromium] Compute the best target node on a GestureTap event
     4        https://bugs.webkit.org/show_bug.cgi?id=85101
     5
     6        Reviewed by Adam Barth.
     7
     8        * features.gypi:
     9        Enable the touch adjustment code by default.
     10        * src/WebInputEventConversion.cpp:
     11        (WebKit::PlatformGestureEventBuilder::PlatformGestureEventBuilder):
     12        In the case of a GestureTap, deltaX represents the horizontal padding of the touch
     13        area and deltaY represents the vertical padding of the touch area.
     14
    1152012-05-14  Adrienne Walker  <enne@google.com>
    216

  • trunk/Source/WebKit/chromium/features.gypi

    r116815 r116983  
    9898      'ENABLE_SVG=<(enable_svg)',
    9999      'ENABLE_SVG_FONTS=<(enable_svg)',
     100      'ENABLE_TOUCH_ADJUSTMENT=1',
    100101      'ENABLE_TOUCH_EVENTS=<(enable_touch_events)',
    101102      'ENABLE_TOUCH_ICON_LOADING=<(enable_touch_icon_loading)',

  • trunk/Source/WebKit/chromium/src/WebInputEventConversion.cpp

    r116815 r116983  
    149149    case WebInputEvent::GestureTap:
    150150        m_type = PlatformEvent::GestureTap;
     151        m_area = IntSize(e.deltaX * 2, e.deltaY * 2);
    151152        break;
    152153    case WebInputEvent::GestureTapDown:
Note: See TracChangeset for help on using the changeset viewer.