Created attachment 463607 [details]
Patch

Comment on attachment 463607 [details]
Patch

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

> Source/WebCore/layout/integration/inline/LayoutIntegrationLineLayout.cpp:199
> +        auto listMarkerIsNested = enclosingListItem != associatedListItem;

Can these both be null? I notice we are using pointers for both.

(In reply to Darin Adler from comment #2)
> Comment on attachment 463607 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=463607&action=review
> 
> > Source/WebCore/layout/integration/inline/LayoutIntegrationLineLayout.cpp:199
> > +        auto listMarkerIsNested = enclosingListItem != associatedListItem;
> 
> Can these both be null? I notice we are using pointers for both.
No, we don't expect neither of them null during layout (that would indicate a detached subtree).

Committed 256858@main (7a26f091845d): <https://commits.webkit.org/256858@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 463607 [details].

<rdar://problem/102532269>