Bug 13395 – REGRESSION: Overlapping text when viewing Yahoo! Mail message (due to use of naturalHeight/Width)

• Component: Layout and Rendering
• Status: RESOLVED DUPLICATE

[David Kilzer (:ddkilzer)]

* SUMMARY
Overlapping text is drawn when viewing certain messages through Yahoo! Mail.

* STEPS TO REPRODUCE
1. Open Safari/WebKit.
2. Open attached test case (after unarchiving) in Safari/WebKit.
3. If overlapping text is not seen, reload page (once or twice).

* EXPECTED RESULTS
The page should render without overlapping text.

* ACTUAL RESULTS
The page renders with overlapping text.

* REGRESSION
This is a regression from shipping Safari 2.0.4 (419.3) on Mac OS X 10.4.9 (8P135).  Seen with a local debug build of WebKit r20930.

* NOTES
See attached screenshot and test case.  My apologies for not reducing this test case first.  (Hopefully there are no Yahoo! sekrits in the page!)

[David Kilzer (:ddkilzer)]

Created attachment 14075 [details]
Unreduced test case

Unreduced test case.

[David Kilzer (:ddkilzer)]

Created attachment 14076 [details]
Screenshot

Screenshot of issue.

The imgsDone function on the page, intended for use with Firefox, always returns 'true' in WebKit since naturalWidth and naturalHeight are not supported. As a result, the rmvScroll function proceeds to adjust the layout based on incorrect image dimensions. This goes unnoticed in Safari 2.0.4 because of a JavaScript error (probably due to incomplete CSS object model support), but in TOT rmvScroll completes successfully, which results in the incorrect layout.

If bug 13333 is not addressed, Yahoo will need to use a different technique to tell if images have loaded.

[David Kilzer (:ddkilzer)]

Fixed by Bug 13333.



*** This bug has been marked as a duplicate of 13333 ***