Closed Bug 449362 Opened 14 years ago Closed 14 years ago
4.35 KB, image/png
1.18 KB, text/html
14.01 KB, patch
|Details | Diff | Splinter Review|
1.49 KB, text/html
1.22 KB, text/html
Uploaded a screenshot that shows the observed behaviour. Note that the act of taking a screenshot got the page to re-render itself correctly, so I doctored the image to show what it looks like before the re-render.
Component: General → Layout: Tables
Product: Firefox → Core
QA Contact: general → layout.tables
I was not able to reproduce the problem on Windows XP, so it seems to be Linux-only.
The behaviour was originally observed on both Ubuntu Linux and Windows XP in a web app. I used my Linux box to reduce the page down to the simplest possible static html that would reproduce the issue. Unfortunately this page is only broken for Linux. So the problem is NOT Linux-only. It is just that the example page only reproduces the problem on Linux.
Component: Layout: Tables → General
Product: Core → Firefox
Version: unspecified → 3.0 Branch
This updated example displays the buggy behaviour on Ubuntu Linux, Win XP, and Mac OSX.
Attachment #332506 - Attachment is obsolete: true
FYI, the reason the original example was OS-dependent seems to be that the text was changing in the second column, and so that column could change widths, depending on the font of the text. Also the URL listed above has been updated to the OS-independant version.
(In reply to comment #5) > Created an attachment (id=332655) [details] Ok, thanks. Now I can indeed reproduce it on Windows. Works: 2008020613 Fails: 2008020614 Regression window is: http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&date=explicit&mindate=2008-02-06+12%3A00&maxdate=2008-02-06+15%3A00 Probably caused by Bug 414298.
Pushed as 17140:7e87332a44d4.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1a2
Comment on attachment 334517 [details] [diff] [review] This fixes things We should probably fix this on the branch too...
Need to add a test once we have a way to test invalidation.
Depends on: 451332
Flags: blocking1.9.1? → in-testsuite?
Boris, I'd like to let this bake a couple of days and approve it for landing (in 22.214.171.124) on Monday. Will you have time to land it then?
Yeah, that should be fine.
Comment on attachment 334517 [details] [diff] [review] This fixes things Approved for 126.96.36.199. Please land in CVS. a=ss
Fixed on branch.
Verified for 3.02 with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:188.8.131.52) Gecko/2008090212 Firefox/3.0.2.
Pushed those tests as http://hg.mozilla.org/mozilla-central/rev/80abbea018b2 Sorry for the lag on that, and thanks for the tests!
Flags: in-testsuite? → in-testsuite+
Hmm. That test actually makes no sense, and will fail randomly (and just did, in one of my try server runs). I'll try to fix it.
Pushed http://hg.mozilla.org/mozilla-central/rev/72e869771e19 to address that
You need to log in before you can comment on or make changes to this bug.