May as well get a proper review of this patch, now with tests. The one other question I have is whether it makes sense to skip independent formatting contexts. I think it kind of does (it definitely does if the formatting context is something non-block, like a flex or grid container), but it would also mean skipping over an `overflow: hidden` child of a flex item in a -webkit-line-clamp controlled flex container, which is not something that Blink does.
Bug 866102 Comment 46 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
May as well get a proper review of an updated patch, now with tests. The one other question I have is whether it makes sense to skip independent formatting contexts. I think it kind of does (it definitely does if the formatting context is something non-block, like a flex or grid container), but it would also mean skipping over an `overflow: hidden` child of a flex item in a -webkit-line-clamp controlled flex container, which is not something that Blink does.