Bug 15060

Summary:

REGRESSION: Misaligned images at http://guide.apple.com/

Product:

WebKit

Reporter:

Adele Peterson <adele>

Component:

Images

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED INVALID

Severity:

Normal

CC:

bdakin, hyatt, mitz

Priority:

Keywords:

InRadar

Version:

523.x (Safari 3)

Hardware:

Mac

OS:

OS X 10.4

URL:

http://guide.apple.com

Attachments:

Description	Flags
test case	none
screenshot	none
Reduction as rendered in TOT	none
Reduction as rendered in older version	none

Adele Peterson

Reported 2007-08-23 10:48:21 PDT

8/22/07 11:14 PM Chris Petersen: * SUMMARY With TOT (25192, 14864) running under Leopard9A529, I'm noticing images are misaligned in a TABLE at http://guide.apple.com/. See screen shot. This problem doesn't happen with Safari running under stock webkit on 9A529. * STEPS TO REPRODUCE 1. With TOT (25192, 14864) running under Leopard9A529, go to http://guide.apple.com/ 2. Notice the image shifted slightly in the "Search by Keyword" and "Browse by Category" section * RESULTS No image shifting should see in the table but does 8/22/07 11:25 PM Chris Petersen: Semi reduced test case has been attached. 8/23/07 12:13 AM Chris Petersen: Here is the HTML structure. The problem seems only happen if the table is nested in a <center> element/ <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> <html> <head> <base href="http://guide.apple.com/"> <title>Apple - Macintosh Products Guide</title> </head> <body> <center> <table width="180" border="0" cellpadding="0" cellspacing="0"> <tr> <td width="8"><img src="/catbar/boxtopleftcorner08222002.gif"></td> <td><img src="/catbar/boxtop08222002.gif"></td> </tr> <tr> <td background="/catbar/boxtopleft08222002.gif"></td> <td>me</td> </tr> </table> </center> </body> </html> 8/23/07 12:17 AM Chris Petersen: The only table specific changes I see have been these two: http://trac.webkit.org/projects/webkit/changeset/25174 http://trac.webkit.org/projects/webkit/changeset/25171

Attachments
test case (530 bytes, text/html) 2007-08-23 10:49 PDT, Adele Peterson	no flags	Details
screenshot (224.56 KB, image/tiff) 2007-08-23 11:21 PDT, Adele Peterson	no flags	Details
Reduction as rendered in TOT (855 bytes, image/png) 2007-08-24 14:34 PDT, mitz	no flags	Details
Reduction as rendered in older version (859 bytes, image/png) 2007-08-24 14:35 PDT, mitz	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Adele Peterson

Comment 1 2007-08-23 10:48:53 PDT

<rdar://problem/5431566>

Adele Peterson

Comment 2 2007-08-23 10:49:20 PDT

Created attachment 16097 [details] test case

mitz

Comment 3 2007-08-23 11:10:21 PDT

Hmm... DOM and render trees are the same. Looks like an image tiling problem. Adele/Chris, can you check if it regressed in <http://trac.webkit.org/projects/webkit/changeset/25187> (maybe also check Tiger)?

Adele Peterson

Comment 4 2007-08-23 11:21:00 PDT

Created attachment 16098 [details] screenshot

Adele Peterson

Comment 5 2007-08-23 11:32:54 PDT

I can confirm that this was caused by the image tiling change in r25187

mitz

Comment 6 2007-08-23 14:06:17 PDT

Simpler: <div style="width: 100px; height: 100px; background: url(http://guide.apple.com//catbar/boxtopleft08222002.gif)"></div>

mitz

Comment 7 2007-08-23 14:28:01 PDT

I am pretty sure the bug is not in WebKit. Using a different GIF with the same dimensions makes it go away.

Adele Peterson

Comment 8 2007-08-24 14:04:20 PDT

So now I can reproduce on the live site if i resize the window. Mitz, I'm not sure I understand your simpler reduction. I don't see a difference between TOT and an older version of Safari. How does the bug appear with that reduction?

mitz

Comment 9 2007-08-24 14:30:46 PDT

Created attachment 16111

Adele Peterson

Comment 10 2007-08-24 14:34:02 PDT

it just shows up as single lines for me...but I'm on Leopard. Tricky.

mitz

Comment 11 2007-08-24 14:34:58 PDT

Created attachment 16112 [details] Reduction as rendered in TOT

mitz

Comment 12 2007-08-24 14:35:33 PDT

Created attachment 16113 [details] Reduction as rendered in older version

Adele Peterson

Comment 13 2007-08-27 13:49:31 PDT

Mitz was right that the bug was in a lower component. Marking as invalid. This will be fixed internally.

Note You need to log in before you can comment on or make changes to this bug.