766907 - Do we need to traverse frames in overflow lines in nsBidiPresUtils::Resolve?

Status: UNCONFIRMED

(Core :: Layout, defect)

Description

[Mats Palmgren (inactive)]

Follow-up on review comment by Simon Montagu 2012-06-19 22:54:21 PDT:

::: layout/base/nsBidiPresUtils.cpp
@@ +607,5 @@
>      nsBlockInFlowLineIterator lineIter(block, block->begin_lines());
>      bpd.mPrevFrame = nsnull;
>      bpd.GetSubParagraph()->mPrevFrame = nsnull;
>      TraverseFrames(aBlockFrame, &lineIter, block->GetFirstPrincipalChild(), &bpd);
> +    // XXX what about overflow lines?

So what about them? do we need to add something like
 |TraverseFrames(aBlockFrame, &lineIter, block->GetFirstChild(kOverflowList), &bpd);|
here?