827915 - Get rid of SVG's own invalidation mechanisms and rely on DLBI instead

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Jonathan Watt [:jwatt]]

+++ This bug was initially created as a clone of Bug #802628 +++

[Bug 802628 became about a subpart of the work for this bug, so I've cloned it to this bug which will be about finishing that work.]

SVG still has its own invalidation mechanisms after landing of DLBI (bug 539356). There are a bunch of SVG invalidation bugs that were caused by the landing of DLBI, mostly because DLBI and the existing SVG invalidation mechanisms don't play nicely together. Ultimately we want SVG to solely use DLBI for invalidation, and it's probably easier to implement that than to fix all the DLBI SVG invalidation regressions individually. This bug is for getting rid of the SVG invalidation mechanisms and converting SVG to use DLBI.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Remove the remaining dead code.

Comment 2

[Daniel Holbert [:dholbert]]

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

I found two more references to this function (in comments), with a MXR search:
http://mxr.mozilla.org/mozilla-central/source/layout/svg/nsSVGTextFrame2.cpp?rev=d9b15dac4de4#4750
http://mxr.mozilla.org/mozilla-central/source/layout/svg/nsSVGUtils.cpp?rev=747173e387c9#582

r=me with those comments corrected.

Comment 3

[Jonathan Watt [:jwatt]]

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

(In reply to Daniel Holbert [:dholbert] from comment #2)
> I found two more references...

I previously filed bug 877429 and bug 877432 on those. As agreed on IRC, landing without removing those instances.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

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