Bug 13395

Summary:

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

Product:

WebKit

Reporter:

David Kilzer (:ddkilzer) <ddkilzer>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED DUPLICATE

Severity:

Normal

CC:

andersca, mitz

Priority:

Keywords:

Regression

Version:

523.x (Safari 3)

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
Unreduced test case	none
Screenshot	none

Description David Kilzer (:ddkilzer) 2007-04-18 09:57:12 PDT

* 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!)

Comment 1 David Kilzer (:ddkilzer) 2007-04-18 09:58:23 PDT

Created attachment 14075 [details]
Unreduced test case

Unreduced test case.

Comment 2 David Kilzer (:ddkilzer) 2007-04-18 09:59:24 PDT

Created attachment 14076 [details]
Screenshot

Screenshot of issue.

Comment 3 mitz 2007-04-18 12:40:29 PDT

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.

Comment 4 David Kilzer (:ddkilzer) 2007-04-19 13:34:22 PDT

Fixed by Bug 13333.



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