Open Bug 766907 Opened 12 years ago Updated 2 years ago

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

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Type:
defect

Tracking

()

Status:
UNCONFIRMED

People

(Reporter: MatsPalmgren_bugz, Unassigned)

Details

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?
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.