Bug 20324

Summary: A change in SVG Glyph wont show up
Product: WebKit Reporter: Tuom Larsen <tuom.larsen>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WORKSFORME    
Severity: Major    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
Attachments:
Description Flags
Testcase
none
First attempt
none
Attachment none

Description Tuom Larsen 2008-08-07 16:36:11 PDT 
When SVG Font is defined and the attribute "d" of one of its glyphs is changed after the page loads, the current rendering wont reflect the change.
Comment 1 Tuom Larsen 2008-08-07 16:36:58 PDT 
Created attachment 22701 [details]
Testcase
Comment 2 Rob Buis 2008-08-24 04:54:23 PDT 
Created attachment 22961 [details]
First attempt

Simple patch that just takes into account d attribute changes + testcase.
Cheers,

Rob.
Comment 3 Rob Buis 2008-08-30 10:42:23 PDT 
Was landed in r35905.
Comment 4 Tuom Larsen 2008-10-10 15:12:04 PDT 
Hello,
sorry to reopen the bug, but it started to crash in the nighlies, r37458 on win.
I just try to open the testcase and it crashes.
Comment 5 Darin Adler 2008-10-12 17:01:55 PDT 
Comment on attachment 22961 [details]
First attempt

Clearing the review flag on the patch because the patch was landed.
Comment 6 Tuom Larsen 2008-10-13 14:58:25 PDT 
(In reply to comment #4)
> sorry to reopen the bug, but it started to crash in the nighlies, r37458 on
> win.

I downgraded to Safari 3.1.2, and running with r37458 it doesn't crash anymore.

However, it doesn't render correctly. The shape of the glyph isn't updated until I select the text. Please, see the attachment.
Comment 7 Tuom Larsen 2008-10-13 14:59:32 PDT 
Created attachment 24328 [details]
Attachment
Comment 8 Eric Seidel (no email) 2008-10-21 11:48:40 PDT 
Works for me with the latest nightly.  If you have a crashing bug in the latest nightly please file a separate bug with test case.
Comment 9 Eric Seidel (no email) 2008-10-21 11:52:00 PDT 
The shape of the glyph is updating correctly for me using:
Version 3.1.2 (5525.20.1)
and a locally built version of WebKit r37761.