375497 - Line break incorrectly allowed between text and span

Status: VERIFIED WORKSFORME

(Core :: Layout: Block and Inline, defect)

Description

[Jesse Ruderman]

Attachment: testcase (there should be no line break between "a" and "b")

When this page is first loaded, it is line-broken between "a" and "b".  If I remove the body and put it back, the line break goes away.  The line break also goes away if I put a div around it all.  I chose the latter for the reference so the reftest pair could be free of scripts.

(This testcase is based on reftests/text-transform/capitalize-2.html, of all things.  The reduced testcase doesn't involve text-transform at all; it just happens that that reftest lacks a <div> or <p> tag.)

Comment 1

[Jesse Ruderman]

Attachment: reference (with div)

Comment 2

[Boris Zbarsky [:bzbarsky]]

Is the issue being "naked in body"?  That is, if you stick a width style on the <div>, does anything change?  If you put a flush between the text and the <span> in the <div>, does anything change?

Comment 3

[Jesse Ruderman]

> Is the issue being "naked in body"?  That is, if you stick a width style on the
> <div>, does anything change?

Oops.  The bug doesn't require being "naked in body", just being in a block with the right width.  I should have tested that.

> If you put a flush between the text and the
> <span> in the <div>, does anything change?

Flushing *before* "a" fixes it, but flushing between "a" and the span does not.

Comment 4

[Jesse Ruderman]

Is bug 375827 related?

Comment 5

[Boris Zbarsky [:bzbarsky]]

Actually, I was wondering whether flushing between would make it appear even in the non-buggy testcase.  And yes, bug 375827 might be related.

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I don't see this bug on my Mac near-trunk build.

Comment 7

[Jesse Ruderman]

WFM too.  Should the reftest pair be added to the test suite?

Comment 8

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I don't we should add this to reftests.

Comment 9

[Carsten Book [:Tomcat]]

verified fixed using Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9b3pre) Gecko/2007123104 Minefield/3.0b3pre and the testcase from this bug -