[webkit-reviews] review granted: [Bug 216502] [LFC][FFC] Block box flex items establish block formatting contexts : [Attachment 408760] Patch

[bugzilla-daemon at webkit.org]

Simon Fraser (smfr) <simon.fraser at apple.com> has granted zalan
<zalan at apple.com>'s request for review:
Bug 216502: [LFC][FFC] Block box flex items establish block formatting contexts
https://bugs.webkit.org/show_bug.cgi?id=216502

Attachment 408760: Patch

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




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

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

> Source/WebCore/layout/layouttree/LayoutBox.h:108
>      bool isBlockLevelBox() const;
> +    bool isBlockBox() const;

I think a comment is justified to explain the different between BlockLevelBox
and BlockBox