Bug 94978 - [chromium] WebViewImpl::enableTouchHighlight() should always clear any existing highlight when invoked.
Summary: [chromium] WebViewImpl::enableTouchHighlight() should always clear any existi...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: W. James MacLean
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-24 15:39 PDT by W. James MacLean
Modified: 2012-08-24 16:50 PDT (History)
1 user (show)

See Also:

Attachments
Patch (15.50 KB, patch)
2012-08-24 15:41 PDT, W. James MacLean 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description W. James MacLean 2012-08-24 15:39:20 PDT 
[chromium] WebViewImpl::enableTouchHighlight() should always clear any existing highlight when invoked.
Comment 1 W. James MacLean 2012-08-24 15:41:31 PDT 
Created attachment 160513 [details]
Patch

Currently, if GestureTapDown occurs while a highlight is being displayed, and the new event does not generate a new target to highlight, we fail to cancel the existing highlight. This CL address this issue.
Comment 2 WebKit Review Bot 2012-08-24 16:50:02 PDT 
Comment on attachment 160513 [details]
Patch

Clearing flags on attachment: 160513

Committed r126654: <http://trac.webkit.org/changeset/126654>
Comment 3 WebKit Review Bot 2012-08-24 16:50:04 PDT 
All reviewed patches have been landed.  Closing bug.