Commit ef70f41c authored by eric@webkit.org's avatar eric@webkit.org
Browse files

2010-04-27 Ben Murdoch <benm@google.com>

        Reviewed by Simon Hausmann.

        The TouchStationary state of WebCore::PlatformTouchPoint is not handled inside the touch event handler.
        https://bugs.webkit.org/show_bug.cgi?id=37609

        No new tests required, only updating documentation.

        * page/EventHandler.cpp:
        (WebCore::EventHandler::handleTouchEvent): Improve the documentation for handling
            the TouchStationary state as a result of discussions in bug 37609.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@58323 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent cf7ae686
2010-04-27 Ben Murdoch <benm@google.com>
Reviewed by Simon Hausmann.
The TouchStationary state of WebCore::PlatformTouchPoint is not handled inside the touch event handler.
https://bugs.webkit.org/show_bug.cgi?id=37609
No new tests required, only updating documentation.
* page/EventHandler.cpp:
(WebCore::EventHandler::handleTouchEvent): Improve the documentation for handling
the TouchStationary state as a result of discussions in bug 37609.
2010-04-27 Simon Fraser <simon.fraser@apple.com>
 
Reviewed by Darin Adler, Maciej Stachowiak.
......@@ -2748,6 +2748,13 @@ bool EventHandler::handleTouchEvent(const PlatformTouchEvent& event)
touches->append(touch);
// Now build up the correct list for changedTouches.
// Note that any touches that are in the TouchStationary state (e.g. if
// the user had several points touched but did not move them all) should
// only be present in the touches list. They may also be added to the
// targetTouches list later, but should never be in the changedTouches
// list so we do not handle them explicitly here.
// See https://bugs.webkit.org/show_bug.cgi?id=37609 for further discussion
// about the TouchStationary state.
if (point.state() == PlatformTouchPoint::TouchReleased)
releasedTouches->append(touch);
else if (point.state() == PlatformTouchPoint::TouchCancelled)
......
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