Bug 155519

Summary: Follow up to r195769
Product: WebKit Reporter: Enrica Casucci <enrica>
Component: WebKit2Assignee: Enrica Casucci <enrica>
Severity: Normal CC: sam, thorton
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch thorton: review+

Description Enrica Casucci 2016-03-15 16:25:58 PDT
For some types of data detector links we choose not to perform the default action. Therefore we don't generate a synthetic click on tap.
There are to code paths that lead to calling handleSyntheticClick that need to be guarded with this logic.
Only one was covered in r195769.

Comment 1 Enrica Casucci 2016-03-15 16:33:21 PDT
Created attachment 274152 [details]
Comment 2 Tim Horton 2016-03-15 16:44:43 PDT
Comment on attachment 274152 [details]

Is this testable?
Comment 3 Enrica Casucci 2016-03-15 16:57:24 PDT
(In reply to comment #2)
> Comment on attachment 274152 [details]
> Patch
> Is this testable?
I wouldn't know how. The two code paths are triggered depending on the duration of the tap vs touch.
Comment 4 Enrica Casucci 2016-03-15 17:02:04 PDT
Committed revision 198245.