DescriptionRevert of Insert an anonymous inline-table when inserting a table part under an inline. (patchset #3 id:40001 of https://codereview.chromium.org/895953003/)
Reason for revert:
Caused bug 458184, and more. I'll reland a new CL for this as soon as I figure out how to do so without breaking stuff.
I think what needs to be fixed is to add some logic in RenderInline::addChild() / addChildBlahblah(), to drill down to the actual parent of |beforeChild|, if specified. Looks like anonymous inline-level boxes is a rather unknown concept in the engine.
Original issue's description:
> Insert an anonymous inline-table when inserting a table part under an inline.
>
> The spec says that if the parent of the table part is an inline, we must wrap
> it inside an inline-table instead of a table.
>
> Apart from giving correct rendering, this change also helps avoid establishing
> (incorrect) inline continuations in some cases.
>
> BUG=327832, 452591, 53693, 2032
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=189941
TBR=leviw@chromium.org,jchaffraix@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=327832, 452591, 53693, 2032
Patch Set 1 #
Created: 5 years, 10 months ago
(Patch set is too large to download)
Messages
Total messages: 6 (2 generated)
|