[ContentChangeObserver] Gmail text editing controls require two taps

[zalan]

<rdar://problem/51375055>

[zalan]

Created attachment 371336 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 371336 [details]
Patch

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

> Source/WebCore/page/ios/ContentChangeObserver.cpp:98
> +        return element.Element::willRespondToMouseClickEvents();

Ugh. Would it be nicer to add an argument to willRespondToMouseClickEvents() saying whether it's for the touch callout?

[zalan]

(In reply to Simon Fraser (smfr) from comment #2)
> Comment on attachment 371336 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=371336&action=review
> 
> > Source/WebCore/page/ios/ContentChangeObserver.cpp:98
> > +        return element.Element::willRespondToMouseClickEvents();
> 
> Ugh. Would it be nicer to add an argument to willRespondToMouseClickEvents()
> saying whether it's for the touch callout?
Let me think about it.

[WebKit Commit Bot]

Comment on attachment 371336 [details]
Patch

Clearing flags on attachment: 371336

Committed r246096: <https://trac.webkit.org/changeset/246096>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.