Bug 6375

Summary:

SourceAlpha do not display properly until window is redrawn (resized)

Product:

WebKit

Reporter:

Nefaur Khandker <nefaurk>

Component:

SVG

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

public

Priority:

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
Firefox logo in SVG format	none
A screenshot of Safari incorrectly displaying the SVG document	none
A screenshot of Safari correctly displaying the SVG document	none
testcase showing an issue with sourcealpha and feMerge	none
layouttests patch	eric: review+
patch fixing this issue	eric: review+

Nefaur Khandker

Reported 2006-01-04 13:46:03 PST

Although I'm not sure which part of the document fails to display properly (it may be that certain types of nodes of the document), upon first loading firefox-logo.svg in either Safari or DrawTest (using the latest version of WebKit), only a portion of the document is initially drawn. The portion that is drawn is consistently the same portion of the document. I have noticed several actions that then cause the document to be displayed in its entirety: 1) Resizing the Safari or DrawTest window 2) Minimizing the Safari or DrawTest window, and then reactivating the window. (Please note that while the window is in the dock, the SVG document is still not displayed correctly, and it that it is only displayed properly when the window is reactivated.) 3) In Safari, clicking the Back button, and then the Forward button (although clicking the Reload button causes the document to again display incorrectly) 4) In DrawTest, clicking on a toolbar button

Attachments
Firefox logo in SVG format (44.54 KB, image/svg+xml) 2006-01-04 13:46 PST, Nefaur Khandker	no flags	Details
A screenshot of Safari incorrectly displaying the SVG document (50.63 KB, image/png) 2006-01-04 13:49 PST, Nefaur Khandker	no flags	Details
A screenshot of Safari correctly displaying the SVG document (55.29 KB, image/png) 2006-01-04 13:50 PST, Nefaur Khandker	no flags	Details
testcase showing an issue with sourcealpha and feMerge (689 bytes, image/svg+xml) 2006-01-28 01:36 PST, Alexander Kellett	no flags	Details
layouttests patch (1.34 KB, patch) 2006-01-28 02:06 PST, Alexander Kellett	eric: review+	Details Formatted Diff Diff
patch fixing this issue (1.36 KB, patch) 2006-01-28 02:08 PST, Alexander Kellett	eric: review+	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Nefaur Khandker

Comment 1 2006-01-04 13:46:50 PST

Created attachment 5486 [details] Firefox logo in SVG format

Nefaur Khandker

Comment 2 2006-01-04 13:49:19 PST

Created attachment 5487 [details] A screenshot of Safari incorrectly displaying the SVG document

Nefaur Khandker

Comment 3 2006-01-04 13:50:37 PST

Created attachment 5488 [details] A screenshot of Safari correctly displaying the SVG document

Eric Seidel (no email)

Comment 4 2006-01-15 03:07:05 PST

Something is not being invalidated properly. Possibly related to the gradients. We definitely used to render this perfectly.

Eric Seidel (no email)

Comment 5 2006-01-22 02:31:29 PST

This has to do with gradients failing to render the first pass through. Not sure why yet. This affects lots of SVGs.

Joost de Valk (AlthA)

Comment 6 2006-01-22 04:36:45 PST

Adding Regression keyword.

Joost de Valk (AlthA)

Comment 7 2006-01-22 04:43:08 PST

Upping to P1 because this is a regression.

Eric Seidel (no email)

Comment 8 2006-01-26 16:02:58 PST

This is a fundemental rendering issue. Without a fix for this we would not ship an SVG enabled WebKit. Adding SVGHitList keyword and changing to p2.

Eric Seidel (no email)

Comment 9 2006-01-26 16:04:40 PST

This is a regression from previous versions of WebKit+SVG, but not from a previous shipping version of Safari, hence why I've removed the regression keyword and set only to p2.

Eric Seidel (no email)

Comment 10 2006-01-27 09:17:47 PST

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

Alexander Kellett

Comment 11 2006-01-28 01:36:43 PST

Created attachment 6037 [details] testcase showing an issue with sourcealpha and feMerge fixing the attached testcase also appears to have fixed this issue

Alexander Kellett

Comment 12 2006-01-28 02:06:42 PST

Created attachment 6040 [details] layouttests patch

Alexander Kellett

Comment 13 2006-01-28 02:08:24 PST

Created attachment 6041 [details] patch fixing this issue

Eric Seidel (no email)

Comment 14 2006-01-28 02:16:48 PST

Thanks for another great fix.

Eric Seidel (no email)

Comment 15 2006-01-28 02:19:38 PST

Comment on attachment 6040 [details] layouttests patch Looks good. I tweaked a bit when landing.

Eric Seidel (no email)

Comment 16 2006-01-28 02:19:55 PST

Comment on attachment 6041 [details] patch fixing this issue Looks great.

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