Closed Bug 723376 Opened 13 years ago Closed 4 months ago

Removing and adding SVG causes the style attribute to be serialized and reparsed

Categories

(Core :: SVG, defect)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: jruderman, Unassigned)

References

Details

(Keywords: perf, testcase)

Attachments

(1 file)

Removing and adding SVG causes the style attribute to be serialized and reparsed. This seems inefficient. Combined with serialization bugs such as bug 475216, this can lead to strange things happening!
Yes, SVG does this very very purposefully. See nsSVGElement::BindToTree. We could try to make this better by, say, storing the base URI in the attr value or something, and not doing the reparse if the base URI didn't change.... We wouldn't serialize the CSS decl if we actually have a string it came from, but in this case we don't, since it was built up programmatically.
Keywords: perf
Blocks: refdyn
OS: Mac OS X → All
Hardware: x86_64 → All
The testcase here works for me, but bug 720077 looks to be caused by the reparse.
Blocks: 720077
Severity: normal → S3

We don't do this any more.

Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → WORKSFORME
Depends on: 1494356
Resolution: WORKSFORME → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: