[webkit-reviews] review granted: [Bug 221275] REGRESSION(r272128): ASSERTION FAILED: run.inlineItem.isText() || run.inlineItem.isInlineBoxStart() || run.inlineItem.isInlineBoxEnd() on fast/layoutformattingcontext/table-basic-row-baseline-align.html : [Attachment 419208] 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 221275: REGRESSION(r272128): ASSERTION FAILED: run.inlineItem.isText() ||
run.inlineItem.isInlineBoxStart() || run.inlineItem.isInlineBoxEnd() on
fast/layoutformattingcontext/table-basic-row-baseline-align.html
https://bugs.webkit.org/show_bug.cgi?id=221275

Attachment 419208: Patch

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




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

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

> Source/WebCore/layout/inlineformatting/InlineContentBreaker.cpp:69
> +    // Based on standards commit boundary ruless it would be enough to check
the first inline item, but due to the table quirk, we can have

standard commit boundary rules