[Webkit-unassigned] [Bug 236710] [content-visibility] Add support for css content-visibility: hidden

[bugzilla-daemon at webkit.org]

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

--- Comment #11 from Rob Buis <rbuis at igalia.com> ---
Comment on attachment 455644
  --> https://bugs.webkit.org/attachment.cgi?id=455644
Improve top layer support

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

>> Source/WebCore/dom/Element.cpp:4806
>> +        child.updateContentVisibilityData(shouldSkip, contentVisibility);
> 
> How do you update children on new DOM mutations? Also, shouldn't this use the composed / flattened tree?

What kind of new DOM mutations do you mean? I did some local tests (create whole c-v tree dynamically, just add a text node or element to the c-v subtree) and things seem to work with c-v: hidden. It is possible the state will not be updated (immediately) in such cases, but the painting will stop at a higher level so it should not cause rendering issues.

Having said that, feel free to share some tricky test scenarios :) I am sure more WPT tests here are welcome.

-- 
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/20220330/7f8c3b9c/attachment-0001.htm>