[Webkit-unassigned] [Bug 204578] [css-grid] Failures on css/css-grid/grid-model/grid-box-sizing-001.html
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Feb 2 10:09:54 PST 2021
https://bugs.webkit.org/show_bug.cgi?id=204578
--- Comment #9 from zsun at igalia.com ---
(In reply to Javier Fernandez from comment #7)
> Comment on attachment 418866 [details]
> Patch
>
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=418866&action=review
>
> r=me
>
> > Source/WebCore/ChangeLog:14
> > +
>
> Nit: sometimes it helps to add a line here to describe the change in this
> file.
>
> > Source/WebCore/rendering/RenderGrid.cpp:196
> > + bool hasDefiniteLogicalHeight = RenderBlock::hasDefiniteLogicalHeight() || hasOverridingLogicalHeight() || computeContentLogicalHeight(MainOrPreferredSize, style().logicalHeight(), WTF::nullopt);
>
> Maybe we could add here FIXME comment to suggest the idea of caching the
> hasDefiniteLogicalHeight if this new call causes a relevant performance
> regression.
>
> > Source/WebCore/rendering/RenderGrid.cpp:253
> > +
>
> Remove this empty line, please.
>
> > LayoutTests/imported/w3c/ChangeLog:8
> > + * web-platform-tests/css/css-grid/grid-model/grid-box-sizing-001-expected.txt:
>
> You could mention here that some test cases pass now thanks to this change.
All comments have been addressed. Thank you!
--
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/20210202/6e8484cf/attachment.htm>
More information about the webkit-unassigned
mailing list