Bug 3509

Summary:

REGRESSION(125-412) Portion of nested table is not painted correctly on cafepress site

Product:

WebKit

Reporter:

Chris Petersen <c.petersen87>

Component:

Tables

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ap, bdakin, bugs, ddkilzer, joost, mitz, webkit-bugs

Priority:

Version:

412

Hardware:

Mac

OS:

OS X 10.4

URL:

http://www.cafepress.com/thestudiostore.4699899

Bug Depends on:

Bug Blocks:

4899

Attachments:

Description	Flags
Reduced test case of problem	none
Page screen shot	none
Test case screen shot	none
Test case screen shot - ToT	none
Further reduced testcase	none
Force child repaint if some of its layout was done at an intermediate position	darin: review+

Chris Petersen

Reported 2005-06-13 12:37:22 PDT

Expected Results: No painting issue occurs when table is rendered What I got: A portion of the nested table is not painted correctly. This issue only happens with Version 2.0 (412) since I can't reproduce under Safari 1.2.4 (10.3.8) Steps to reproduce: 1) Under 10.4.1 (8B15), launch Safari Version 2.0 (412) 2) Go to http://www.cafepress.com/thestudiostore.4699899 3) After page is displayed, a portion of the table is painted incorrectly. See attached screen shot of page. 3/7/05 10:35 AM chris petersen: I have created a reduced test case. It contains two tables (one nested is the parent table) using width percentages. Also, the child table is nested in a form element. You should be able to reproduce loading the sample test case. If not, please try several reloads of the page.

Attachments
Reduced test case of problem (530 bytes, text/html) 2005-06-13 12:39 PDT, Chris Petersen	no flags	Details
Page screen shot (535.61 KB, image/jpeg) 2005-06-13 12:45 PDT, Chris Petersen	no flags	Details
Test case screen shot (215.67 KB, image/jpeg) 2005-06-13 12:47 PDT, Chris Petersen	no flags	Details
Test case screen shot - ToT (11.30 KB, image/jpeg) 2005-11-09 12:22 PST, Daniel Udey	no flags	Details
Further reduced testcase (480 bytes, text/html) 2005-12-30 06:46 PST, mitz	no flags	Details
Force child repaint if some of its layout was done at an intermediate position (3.90 KB, patch) 2006-01-21 05:23 PST, mitz	darin: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Chris Petersen

Comment 1 2005-06-13 12:39:18 PDT

Created attachment 2301 [details] Reduced test case of problem

Chris Petersen

Comment 2 2005-06-13 12:40:29 PDT

Apple Bug: <rdar://problem/4039615>

Chris Petersen

Comment 3 2005-06-13 12:45:31 PDT

Created attachment 2302 [details] Page screen shot

Chris Petersen

Comment 4 2005-06-13 12:47:09 PDT

Created attachment 2303 [details] Test case screen shot

Joost de Valk (AlthA)

Comment 5 2005-06-13 23:51:47 PDT

Can't confirm this one i'm afraid, works on 13 june build.

Chris Petersen

Comment 6 2005-06-14 09:52:04 PDT

This issue is still occuring on TOT WebKit (06/13) for me. The best way to reproduce for me is too: Save a copy of test case to local drive Reset Safari and quit Relaunch Safari and drag the test case into the browser window Table isn't correctly rendered (same as screen shot).

Daniel Udey

Comment 7 2005-11-09 12:22:27 PST

Created attachment 4653 [details] Test case screen shot - ToT This bug is still present in ToT Nov 9th. It seems that it can only be reproduced once per browser session - further reloads seemed to render properly, likely making this a tricky bug to track.

Eric Seidel (no email)

Comment 8 2005-12-28 02:23:03 PST

I'm pretty sure this is a dupe of 4899, or rather the other way around. As a regression, this gets a p1.

Eric Seidel (no email)

Comment 9 2005-12-28 02:23:39 PST

*** Bug 4899 has been marked as a duplicate of this bug. ***

mitz

Comment 10 2005-12-30 06:46:53 PST

Created attachment 5384 [details] Further reduced testcase The bug seems to be about "quirk margins".

mitz

Comment 11 2005-12-30 06:47:52 PST

In the "further reduced testcase", you can Select All after clicking test to trigger repainting of the missing piece.

David Kilzer (:ddkilzer)

Comment 12 2006-01-08 09:40:29 PST

Another litmus test for this bug is to hide Safari, then switch back to it to see the correct rendering.

David Kilzer (:ddkilzer)

Comment 13 2006-01-08 09:46:43 PST

*** Bug 6278 has been marked as a duplicate of this bug. ***

mitz

Comment 14 2006-01-21 05:23:14 PST

Created attachment 5807 [details] Force child repaint if some of its layout was done at an intermediate position

Darin Adler

Comment 15 2006-01-21 11:01:18 PST

Comment on attachment 5807 [details] Force child repaint if some of its layout was done at an intermediate position Looks like a great solution to this issue. r=me

Joost de Valk (AlthA)

Comment 16 2006-01-22 04:51:41 PST

Removing keywords cause bug is closed.

Eric Seidel (no email)

Comment 17 2006-01-31 21:20:31 PST

Removing Regression keyword from bugs already fixed.

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