Bug 50680

Summary: WebTouchEvent::isTouchEventType() does not take into account the type TouchCancel
Product: WebKit Reporter: Benjamin Poulain <benjamin>
Component: WebKit2Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Description Benjamin Poulain 2010-12-08 04:38:48 PST 
WebTouchEvent::isTouchEventType() does not take into account the type TouchCancel. This makes the constructor of the class fail in debug due to ASSERT(isTouchEventType(type));.
Comment 1 Benjamin Poulain 2010-12-08 04:50:18 PST 
Created attachment 75891 [details]
Patch
Comment 2 Andreas Kling 2010-12-08 04:52:37 PST 
Comment on attachment 75891 [details]
Patch

Good catch, r=me
Comment 3 WebKit Commit Bot 2010-12-08 17:10:30 PST 
The commit-queue encountered the following flaky tests while processing attachment 75891 [details]:

fast/workers/storage/use-same-database-in-page-and-workers.html
transitions/change-values-during-transition.html

Please file bugs against the tests.  These tests were authored by dumi@chromium.org, jparent@google.com, mihaip@chromium.org, and pol@apple.com.  The commit-queue is continuing to process your patch.
Comment 4 WebKit Commit Bot 2010-12-08 18:16:00 PST 
Comment on attachment 75891 [details]
Patch

Clearing flags on attachment: 75891

Committed r73581: <http://trac.webkit.org/changeset/73581>
Comment 5 WebKit Commit Bot 2010-12-08 18:16:05 PST 
All reviewed patches have been landed.  Closing bug.