Bug 108950
| Summary: | Touch event should be readonly when shouldGesturesTriggerActive() returns false | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Xianzhu Wang <wangxianzhu> |
| Component: | UI Events | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | jamesr, rbyers, tonikitoo |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Xianzhu Wang
In EventHandler.cpp: EventHandler::handleTouchEvent()
#if ENABLE(GESTURE_EVENTS)
if (shouldGesturesTriggerActive())
hitType |= HitTestRequest::ReadOnly;
#endif
There should be a '!' in the if condition.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Xianzhu Wang
After reading carefully bug 96060, I think I was wrong. The intention of the patch was to tigger active for touch events on chromium-android. Sorry for bother.