[Webkit-unassigned] [Bug 236711] [content-visibility] Add support for css content-visibility: auto

[bugzilla-daemon at webkit.org]

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

--- Comment #11 from Rob Buis <rbuis at igalia.com> ---
Comment on attachment 456817
  --> https://bugs.webkit.org/attachment.cgi?id=456817
Handle selection

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

>>> Source/WebCore/rendering/style/RenderStyle.h:543
>>> +    bool containsSize() const { return effectiveContainment().contains(Containment::Size) || skipsContents(); }
>> 
>> Maybe we can also move `skipsContents()` inside `effectiveContainment()`? Not sure why `containsLayout`, `containsStyle`, and `containsPaint` do not check `skipsContents()` like `containsSize`?
> 
> Maybe we can also move `skipsContents()` inside `effectiveContainment()`? Not sure why `containsLayout`, `containsStyle`, and `containsPaint` do not check `skipsContents()` like `containsSize`?

See https://drafts.csswg.org/css-contain/#valdef-content-visibility-auto:
"Turns on layout containment, style containment, and paint containment for the element."
For c-v: hidden all containment is enabled. But actually, I am not sure in our patches c-v: hidden always enables size containment unconditionally....

-- 
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/20220511/1e5b84ce/attachment.htm>