Bug 196259 – [ContentChangeObserver] Skip anonymous renderers when checking for "willRespondToMouseClickEvents"

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[alan]

they don't count.

[alan]

<rdar://problem/49240029>

[alan]

Created attachment 365971 [details]
Patch

[Dean Jackson]

Comment on attachment 365971 [details]
Patch

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

> LayoutTests/fast/events/touch/ios/content-observation/crash-on-anonymous-renderer.html:32
> +    let rect = tapthis.getBoundingClientRect();
> +    let x = rect.left + rect.width / 2;
> +    let y = rect.top + rect.height / 2;

can use const :)

[alan]

(In reply to Dean Jackson from comment #3)
> Comment on attachment 365971 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=365971&action=review
> 
> > LayoutTests/fast/events/touch/ios/content-observation/crash-on-anonymous-renderer.html:32
> > +    let rect = tapthis.getBoundingClientRect();
> > +    let x = rect.left + rect.width / 2;
> > +    let y = rect.top + rect.height / 2;
> 
> can use const :)
Oh I didn't know about it!

[WebKit Commit Bot]

Comment on attachment 365971 [details]
Patch

Clearing flags on attachment: 365971

Committed r243503: <https://trac.webkit.org/changeset/243503>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.