Bug 14927

Summary:

use element in SVG does not render after parent is displayed off and on

Product:

WebKit

Reporter:

Jean-Pierre Fiset <bugzilla>

Component:

SVG

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ahayes, bugzilla

Priority:

Keywords:

HasReduction

Version:

523.x (Safari 3)

Hardware:

Mac

OS:

All

Attachments:

Description	Flags
Simple Test Case	none
First attempt	oliver: review+

Jean-Pierre Fiset

Reported 2007-08-10 08:02:27 PDT

In SVG, if the parent element of a use element is hidden (via display="none") and then made visible again (via display="inline"), the content of the use element fails to render. A similar test case, where the actual elements are used instead of <use> elements, behaves correctly. This problem appears on Safari 3.0.3 and on WebKit night r24944. This problem was also reproduced on MacIntel and Windows XP.

Attachments
Simple Test Case (2.22 KB, image/svg+xml) 2007-08-10 08:03 PDT, Jean-Pierre Fiset	no flags	Details
First attempt (39.20 KB, patch) 2007-08-16 13:04 PDT, Rob Buis	oliver: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Jean-Pierre Fiset

Comment 1 2007-08-10 08:03:58 PDT

Created attachment 15909 [details] Simple Test Case

Rob Buis

Comment 2 2007-08-16 13:04:09 PDT

Created attachment 16000 [details] First attempt This should fix it :) Cheers, Rob.

Oliver Hunt

Comment 3 2007-08-16 23:47:05 PDT

Comment on attachment 16000 [details] First attempt Could you use a rect instead of circle in the test case? otherwise r=me

Rob Buis

Comment 4 2007-08-16 23:53:06 PDT

(In reply to comment #1) > Created an attachment (id=15909) [edit] > Simple Test Case I forgot to say, these test cases are excellent, keep up the good reports! Cheers, Rob.

Rob Buis

Comment 5 2007-08-16 23:55:12 PDT

Landed in r25123.

Jean-Pierre Fiset

Comment 6 2007-08-17 05:38:38 PDT

(In reply to comment #3) > (From update of attachment 16000 [details] [edit]) > Could you use a rect instead of circle in the test case? > > otherwise r=me > Would you like me to submit a new test case or are you stating that, in the future, I should submit test cases with rectangles as opposed to circles? What are the ramifications of using <rect> over <circle> in test cases?

Jean-Pierre Fiset

Comment 7 2007-08-17 12:10:26 PDT

(In reply to comment #4) > (In reply to comment #1) > > Created an attachment (id=15909) [edit] > > Simple Test Case > > I forgot to say, these test cases are excellent, keep up the good reports! > Cheers, > > Rob. > Some credit is due to the organization that pays me to develop an SVG-enabled interactive atlas framework. See <http://nunaliit.org> Thanks for noticing. JP

Oliver Hunt

Comment 8 2007-08-18 02:38:25 PDT

Jean-Pierre, it just makes the pixel tests a bit more robust across platforms, my comment was regarding the automated test that Rob included in his patch :D

Oliver Hunt

Comment 9 2007-08-18 02:39:29 PDT

Jean-Pierre, so your test was perfect for the purpose of finding and fixing the source of the bug :D

Rob Buis

Comment 10 2007-08-19 01:26:47 PDT

Landed in r25123.

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