[webkit-reviews] review granted: [Bug 202610] [LFC] Remove redundant includes. : [Attachment 380276] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Oct 4 21:55:54 PDT 2019


Tim Horton <thorton at apple.com> has granted zalan <zalan at apple.com>'s request
for review:
Bug 202610: [LFC] Remove redundant includes.
https://bugs.webkit.org/show_bug.cgi?id=202610

Attachment 380276: Patch

https://bugs.webkit.org/attachment.cgi?id=380276&action=review




--- Comment #3 from Tim Horton <thorton at apple.com> ---
Comment on attachment 380276
  --> https://bugs.webkit.org/attachment.cgi?id=380276
Patch

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

You know I’m on board :P

> Source/WebCore/layout/FormattingContext.h:52
> +struct HeightAndMargin;

I usually sort the different kinds separately, but I don’t think it’s a rule :)


More information about the webkit-reviews mailing list