Closed
Bug 771795
Opened 12 years ago
Closed 12 years ago
Stop over-invalidating SVG
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: jwatt, Assigned: jwatt)
Details
(Keywords: perf)
Attachments
(1 file)
|
(deleted),
patch
|
heycam
:
review+
|
Details | Diff | Splinter Review |
We are currently massively over invalidating, and no longer need to any more.
|
Assignee | |
Comment 1•12 years ago
|
||
Attachment #639960 -
Flags: review?(dholbert)
|
|
Assignee | |
Updated•12 years ago
|
Summary: Stop over invalidating SVG → Stop over-invalidating SVG
|
||
Comment 2•12 years ago
|
||
Seems a good time to address this comment:
// XXXSDL Let FinishAndStoreOverflow do this.
...or remove it entirely if no longer applicable.
|
|
Assignee | |
Comment 3•12 years ago
|
||
(In reply to Jet Villegas (:jet) from comment #2)
> Seems a good time to address this comment:
> // XXXSDL Let FinishAndStoreOverflow do this.
It's not clear right now how that code should look after Matt's DLBI work, so for now I'd just like to leave it as it is. Besides that, the rapid release process rules demand that I'm able to disable or back out my work if it turns out something bad requires it on branches, so I'd rather leave the "easy disable" route intact.
|
||
Comment 4•12 years ago
|
||
Comment on attachment 639960 [details] [diff] [review]
patch
(Stealing with permission.)
Attachment #639960 -
Flags: review?(dholbert) → review+
|
|
Assignee | |
Comment 5•12 years ago
|
||
Target Milestone: --- → mozilla16
|
||
Comment 6•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•