Stop over-invalidating SVG

RESOLVED FIXED in mozilla16

Status

()

Core
SVG
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jwatt, Assigned: jwatt)

Tracking

({perf})

Trunk
mozilla16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
We are currently massively over invalidating, and no longer need to any more.
(Assignee)

Comment 1

5 years ago
Created attachment 639960 [details] [diff] [review]
patch
Attachment #639960 - Flags: review?(dholbert)
(Assignee)

Updated

5 years ago
Keywords: perf
(Assignee)

Updated

5 years ago
Summary: Stop over invalidating SVG → Stop over-invalidating SVG

Comment 2

5 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

5 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 on attachment 639960 [details] [diff] [review]
patch

(Stealing with permission.)
Attachment #639960 - Flags: review?(dholbert) → review+
(Assignee)

Comment 5

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/fa1bf35aae2e
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/fa1bf35aae2e
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.