271747 - Width in table cells in bottom row sometimes ignored

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect)

Description

[Martijn Wargers (dead)]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a5) Gecko/20041124 Firefox/0.9.1+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a5) Gecko/20041124 Firefox/0.9.1+

The url sometimes doesn't look 'right'. It happens on first load
(shift->reload), and you notice it when you see a repeating of the same
background image (which should not happen).

See upcoming testcase. That testcase uses the Maciej/Hyatt hack (javascript).
Basically what happens with that testcase is that any width set on table cells
in the bottom table row don't apply.

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

Comment 1

[Martijn Wargers (dead)]

Attachment: Testcase

Comment 2

[Martijn Wargers (dead)]

Ok, this got fixed between 2006-12-07 and 2007-12-08, so fixed by the reflow branch landing.

Comment 3

[Martijn Wargers (dead)]

Attachment: reftest

Comment 4

[Martijn Wargers (dead)]

Attachment: reference rendering

Comment 5

[Martijn Wargers (dead)]

Attachment: reftest diff

Comment 6

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

Comment on attachment 261155 [details] [diff] [review]
reftest diff

You might also want to do a second reference (and a second == line in the manifest) where the cells in both rows have width attributes on them.  But r=dbaron either way.

Comment 7

[Martijn Wargers (dead)]

Attachment: reftest2 diff

Ok, done that in this diff. I'll check this in.

Comment 8

[Martijn Wargers (dead)]

RCS file: /cvsroot/mozilla/layout/reftests/bugs/271747-1a.html,v
done
Checking in 271747-1a.html;
/cvsroot/mozilla/layout/reftests/bugs/271747-1a.html,v  <--  271747-1a.html
initial revision: 1.1
done
RCS file: /cvsroot/mozilla/layout/reftests/bugs/271747-1b.html,v
done
Checking in 271747-1b.html;
/cvsroot/mozilla/layout/reftests/bugs/271747-1b.html,v  <--  271747-1b.html
initial revision: 1.1
done
RCS file: /cvsroot/mozilla/layout/reftests/bugs/271747-1-ref.html,v
done
Checking in 271747-1-ref.html;
/cvsroot/mozilla/layout/reftests/bugs/271747-1-ref.html,v  <--  271747-1-ref.htm
l
initial revision: 1.1
done
Checking in reftest.list;
/cvsroot/mozilla/layout/reftests/bugs/reftest.list,v  <--  reftest.list
new revision: 1.289; previous revision: 1.288
done

Reftests checked in.