Bug 20324 - A change in SVG Glyph wont show up
: A change in SVG Glyph wont show up
Product: WebKit
Classification: Unclassified
Component: SVG
: 528+ (Nightly build)
: PC Windows XP
: P2 Major
Assigned To: Nobody
Depends on:
  Show dependency treegraph
Reported: 2008-08-07 16:36 PDT by Tuom Larsen
Modified: 2008-10-21 11:52 PDT (History)
0 users

See Also:

Testcase (1.12 KB, application/xhtml+xml)
2008-08-07 16:36 PDT, Tuom Larsen
no flags Details
First attempt (11.00 KB, patch)
2008-08-24 04:54 PDT, Rob Buis
no flags Details | Formatted Diff | Diff
Attachment (2.99 KB, image/png)
2008-10-13 14:59 PDT, Tuom Larsen
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
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]
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.

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
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]
Comment 8 Eric Seidel 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 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.