738926 - Overflow:auto block doesn't clear a float it can't fit next to

Status: RESOLVED DUPLICATE of bug 478834

(Core :: Layout: Floats, defect)

Description

[Marc42410]

Firefox renders some MSNBC Video pages incorrectly.
i.e)http://video.msnbc.msn.com/msnbc-tv/46843101/#46839110
The video itself shows okay, but the document underneath it is rendered incorrectly.
It shows correctly with IE9 and Opera 11.61.
Tested with Firefox 14.0a1 nightly build on 3/24, 2012.
The version of executive file is 14.0.0.4466

STR
You don't have to do anything special.
Just create a new profile and go to the URL I mentioned above.

Comment 1

[Alice0775 White]

Attachment: reduced html

Comment 2

[Alice0775 White]

Regression window with html5=true
Cannnot reproduce:
http://hg.mozilla.org/mozilla-central/rev/736619af9701
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a1pre) Gecko/20091101 Minefield/3.7a1pre ID:20091101045105
Can reproduce:
http://hg.mozilla.org/mozilla-central/rev/00924c08d33d
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a1pre) Gecko/20091102 Minefield/3.7a1pre ID:20091102045658
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=736619af9701&tochange=00924c08d33d

Triggered by: Bug 501847

Comment 3

[(mostly gone) XtC4UaLL [:xtc4uall]]

Setting the "overflow" Rule to "none" for the "content/transcript" DIVs makes the Rendering equal amongst the Browsers in the reduced Testcase.

Comment 4

[Boris Zbarsky [:bzbarsky]]

So this page has two left floats followed by an overflow:auto div, all inside an inline which is inside a fixed-width container.

All the patch in bug 501847 did was make the floats not be kids of the same block as the overflow-auto div.

Really minimal testcase coming up that shows the real problem here.  Pretty sure this is a duplicate....

Comment 5

[Boris Zbarsky [:bzbarsky]]

Attachment: More reduced HTML

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

See bug 945682 comment 8 through 10 for a bit more analysis.

Comment 10

[Boris Zbarsky [:bzbarsky]]

Actually, looks like my minimal testcase was not quite minimal enough.  You don't even need the "wrapped in another block" bit.  And it's a problem going back forever (e.g. Firefox 2 has it).

Comment 11

[Boris Zbarsky [:bzbarsky]]

Attachment: Really reduced

Comment 14

[Boris Zbarsky [:bzbarsky]]

Mats, do you have time to look into this?

Comment 15

[Mats Palmgren (inactive)]

Not anytime soon, sorry.  I'm busy with implementing Grid layout and
I suspect fixing this bug is harder than it might seem, given that
dbaron analyzed it (comment 6) but didn't provide a fix.

Comment 16

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

fixed by bug 478834