[Webkit-unassigned] [Bug 216355] Don't create event regions when the page has no subscrollers

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Sep 10 08:57:50 PDT 2020


https://bugs.webkit.org/show_bug.cgi?id=216355

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

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

> Source/WebCore/rendering/RenderLayerCompositor.cpp:880
> +        if (scrollingCoordinator && scrollingCoordinator->hasSubscrollers() != hadSubscrollers)
> +            invalidateEventRegionForAllFrames();

I don't like that this triggers a second compositing update just after we've finished one, and in all frames too!

This means that every page with one or more subscrollers now suffers from an extra full layer tree walk in all frames and a second compositing update.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20200910/a5fa14d0/attachment.htm>


More information about the webkit-unassigned mailing list