Bug 20324 - A change in SVG Glyph wont show up
: A change in SVG Glyph wont show up
Status: RESOLVED WORKSFORME
: WebKit
SVG
: 528+ (Nightly build)
: PC Windows XP
: P2 Major
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2008-08-07 16:36 PST by
Modified: 2008-10-21 11:52 PST (History)


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


Note

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


Description From 2008-08-07 16:36:11 PST
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 From 2008-08-07 16:36:58 PST -------
Created an attachment (id=22701) [details]
Testcase
------- Comment #2 From 2008-08-24 04:54:23 PST -------
Created an attachment (id=22961) [details]
First attempt

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

Rob.
------- Comment #3 From 2008-08-30 10:42:23 PST -------
Was landed in r35905.
------- Comment #4 From 2008-10-10 15:12:04 PST -------
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 From 2008-10-12 17:01:55 PST -------
(From update of attachment 22961 [details])
Clearing the review flag on the patch because the patch was landed.
------- Comment #6 From 2008-10-13 14:58:25 PST -------
(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 From 2008-10-13 14:59:32 PST -------
Created an attachment (id=24328) [details]
Attachment
------- Comment #8 From 2008-10-21 11:48:40 PST -------
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 From 2008-10-21 11:52:00 PST -------
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.