253701 - Table column appears twice or three times

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect)

Description

[osavill]

User-Agent:       Mozilla/5.0 (compatible; Konqueror/3.2; Linux) (KHTML, like Gecko)
Build Identifier: 

http://www.sterlingxs.co.uk/ and 
http://www.sterlingxs.co.uk/scpages/newlaptops.html both show odd behaviour 
when it comes to rendering the left hand side. There appears to be duplication 
of the left hand edge of the main display area. This duplicated area 
disappears when the browser window is made small enough. 
 
http://www.sterlingxs.co.uk/scpages/laptopaccessories.html does not display 
this behaviour. 

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Comment 1

[Boris Zbarsky [:bzbarsky]]

That table cell has a width of "149", but we're rendering it a lot wider than
that, so the background tiles...

Comment 2

[Boris Zbarsky [:bzbarsky]]

Need a minimal testcase...

Comment 3

[Mats Palmgren (inactive)]

Attachment: Testcase

Comment 4

[Mats Palmgren (inactive)]

We have the same layout all the way back to and including Mozilla 1.0.2
It sure seems like a bug to me though.

Comment 5

[Mats Palmgren (inactive)]

Attachment: Testcase #2 (standards mode)

The problem does not occur in Standards mode...

Comment 6

[Boris Zbarsky [:bzbarsky]]

Bernd, it looks like the quirky code in question is at
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/html/table/src/BasicTableLayoutStrategy.cpp&rev=3.225&mark=1071-1077#1070

It looks like this is blowing away all the specified widths, not just that for
the cell in the first row?

Comment 7

[Martijn Wargers (dead)]

I think bug 269156 is the same bug.
Bernd is pointing at the same lines of code there as Boris.

Comment 8

[Boris Zbarsky [:bzbarsky]]

Yeah.  The patches there will probably fix this bug, as well as dozens of other
cell-sizing bugs we have on file....

Comment 9

[j.j.]

WFM with Minefield 3a1. likly fixed with reflow bug 3000330.

Comment 10

[Mats Palmgren (inactive)]

Bug occurs in SeaMonkey 2006120701 (pre reflow branch) on Linux.
Bug does not occur in SeaMonkey 2006120801 (post reflow branch) on Linux.

-> FIXED (by the reflow branch landing)

Comment 11

[Ryan VanderMeulen [:RyanVM]]

Attachment: Reftests

Comment 12

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

Comment on attachment 257621 [details] [diff] [review]
Reftests

Checked in after testing in pre- and post-reflow-branch builds.