177442 - white-space: pre and bidi results in duplicated text frames

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, defect)

Description

[Hixie (not reading bugmail)]

For some reason, the text frames in the following test case get duplicated:
   http://www.hixie.ch/tests/adhoc/css/box/inline/bidi/005.html
(Look at the second box, not the first, which is just the reference rendering.)

ACTUAL RESULTS:
   abckdefgfhidejkblm

EXPECTED RESULTS
   abcdefghijklm

Note: we get other parts of the test wrong too, however, that's not what this
report is about.

We could do with a more reduced test case.

Comment 1

[Simon Montagu :smontagu]

Attachment: Reduced testcase

Comment 2

[Simon Montagu :smontagu]

Attachment: Patch v.1

This patch just copies IBMBIDI code from the "normal" codepath to the
"preformatted" codepath.

We probably want to add something similar in ScanPreData_B as well.

Comment 3

[Simon Montagu :smontagu]

Taking.

Comment 4

[Simon Montagu :smontagu]

Attachment: Testcase for ScanPreData_B

This testcase exposes additional bugs in both the normal and preformatted
cases.

Comment 5

[Simon Montagu :smontagu]

Comment on attachment 104705 [details] [diff] [review]
Patch v.1

This isn't complete enough for review.

Comment 6

[Simon Montagu :smontagu]

*** Bug 301889 has been marked as a duplicate of this bug. ***

Comment 7

[Uri Bernstein]

Seeing on OS X => All/All

Comment 8

[Uri Bernstein]

The fix for bug 333769 included the change in Patch v.1, and fixed the original testcases on this bug.
I also can't reproduce the problem with "Testcase for ScanPreData_B" any more, although that probably has to do with unrelated changes made to word selection/navigation.

I'm resolving this bug FIXED, and I suggest that a separate bug would be opened for "ScanPreData_B", if a testcase can be made to show there is still a problem there.

Comment 9

[Uri Bernstein]

*** Bug 221910 has been marked as a duplicate of this bug. ***