|Summary:||REGRESSION (r18574): User profile pages display incorrectly at ebay.ca|
|OS:||OS X 10.4|
Description crastinator 2007-02-14 23:50:44 PST
Running webkit R19630. Any user profile page (the ones that display feedback) display incorrectly. Tried this on several user pages and I get the same thing. This seems to apply only to the .ca version of ebay.
Comment 1 mitz 2007-02-15 04:32:14 PST
Created attachment 13183 [details] Partial reduction The regression happened in r18574 (fix for bug 5164). Changing the percentScaleFactor in Length.h to 1 fixes the bug and this reduction, making me think that the bug has to do with one of the FIXMEs in AutoTableLayout.cpp.
Comment 2 mitz 2007-02-15 06:56:10 PST
Created attachment 13184 [details] Fix table layout bug The actual bug predates r18574 and fixing it gives better results in two existing tests. The reason for the regression is that the higher accuracy made it happen even in this bug's particular case (you can change the yellow cell's width in the reduction from 4 to 20 to get a case that fails in pre-r18574 builds).
Comment 3 David Kilzer (:ddkilzer) 2007-02-15 08:30:23 PST
(In reply to comment #2) > Created an attachment (id=13184)  > Fix table layout bug The ChangeLog says that tables/mozilla_expected_failures/core/col_span2.html is "fixed", so should it move out of the mozilla_expected_failures directory?
Comment 4 mitz 2007-02-15 08:34:29 PST
(In reply to comment #3) > The ChangeLog says that tables/mozilla_expected_failures/core/col_span2.html is > "fixed", so should it move out of the mozilla_expected_failures directory? > That is a long test with many individual cases. In some of them WebKit's layout still don't match Firefox's, so I didn't want to move it. Perhaps a more appropriate phrasing for the change log would be "Covered by existing layout tests:".
Comment 5 Darin Adler 2007-02-15 09:46:33 PST
Comment on attachment 13184 [details] Fix table layout bug r=me