Remove code that sets |deltaX| and |deltaY| for WebKit gesture tap events

Once https://bugs.webkit.org/show_bug.cgi?id=92151 lands and is gardened in, WebKit
gesture tap events will use |boundingBox| instead of |deltaX| and |deltaY| to
determine the gesture's touch region, so these special cases may be removed.

BUG=138572


Review URL: https://chromiumcodereview.appspot.com/10830081

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149432 0039d316-1c4b-4281-b951-d872f2087c98
parent ed8111d7
...@@ -419,16 +419,6 @@ class LocalGestureEvent : ...@@ -419,16 +419,6 @@ class LocalGestureEvent :
data().deltaY = details.generic_y(); data().deltaY = details.generic_y();
data().type = ConvertToWebInputEvent(type_); data().type = ConvertToWebInputEvent(type_);
data().boundingBox = details.bounding_box(); data().boundingBox = details.bounding_box();
// 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 { virtual int GetLowestTouchId() const OVERRIDE {
......
...@@ -414,16 +414,6 @@ WebKit::WebGestureEvent MakeWebGestureEventFromAuraEvent( ...@@ -414,16 +414,6 @@ WebKit::WebGestureEvent MakeWebGestureEventFromAuraEvent(
gesture_event.deltaY = event->details().generic_y(); gesture_event.deltaY = event->details().generic_y();
gesture_event.modifiers = EventFlagsToWebEventModifiers(event->flags()); gesture_event.modifiers = EventFlagsToWebEventModifiers(event->flags());
// 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 (event->type() == ui::ET_GESTURE_TAP) {
gesture_event.deltaX = event->details().bounding_box().width() / 2;
gesture_event.deltaY = event->details().bounding_box().height() / 2;
}
return gesture_event; return gesture_event;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment