771795 - Stop over-invalidating SVG

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Jonathan Watt [:jwatt]]

We are currently massively over invalidating, and no longer need to any more.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 2

[Jet Villegas (inactive)]

Seems a good time to address this comment:
// XXXSDL Let FinishAndStoreOverflow do this.

...or remove it entirely if no longer applicable.

Comment 3

[Jonathan Watt [:jwatt]]

(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

[Cameron McCormack (:heycam)]

Comment on attachment 639960 [details] [diff] [review]
patch

(Stealing with permission.)

Comment 5

[Jonathan Watt [:jwatt]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fa1bf35aae2e

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/fa1bf35aae2e