[webkit-reviews] review granted: [Bug 191712] [iOS] 2 subsequent taps are required to trigger certain tasks on the desktop version of YouTube.com : [Attachment 354985] Patch

[bugzilla-daemon at webkit.org]

Simon Fraser (smfr) <simon.fraser at apple.com> has granted zalan
<zalan at apple.com>'s request for review:
Bug 191712: [iOS] 2 subsequent taps are required to trigger certain tasks on
the desktop version of YouTube.com
https://bugs.webkit.org/show_bug.cgi?id=191712

Attachment 354985: Patch

https://bugs.webkit.org/attachment.cgi?id=354985&action=review




--- Comment #3 from Simon Fraser (smfr) <simon.fraser at apple.com> ---
Comment on attachment 354985
  --> https://bugs.webkit.org/attachment.cgi?id=354985
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=354985&action=review

> Source/WebCore/ChangeLog:3
> +	   [iOS] 2 subsequent taps are required to trigger certain tasks on the
desktop version of YouTube.com

Maybe talk about :hover

> Source/WebCore/ChangeLog:10
> +	   The idea here is that if the new content (result of dispatching
mouseMoved) does not respond to mouse click events, it is most likely only for
tooltip-like reasons.
> +	   Hover-like drop down menus tend to respond to mouse click events.

I think this should mention "content observation" or WKContentObservation