1450705 - Don't call ResolveTag to see if our parent is SVG or not.

Status: RESOLVED FIXED

(Core :: Layout, task)

Description

[Emilio Cobos Álvarez [:emilio]]

We don't extend svg elements, except in a lone test.

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1450705: Don't go through the binding manager to see if our parent is an SVG element.

We don't extend svg elements, except in a lone test, that isn't really impacted
by this.

I agree this should look at the frame btw, though that looks a bit out of scope
for this bug.

Review commit: https://reviewboard.mozilla.org/r/233022/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233022/

Comment 2

[Emilio Cobos Álvarez [:emilio]]

Comment on attachment 8964305 [details]
Bug 1450705: Don't go through the binding manager to see if our parent is an SVG element.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/233022/diff/1-2/

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8964305 [details]
Bug 1450705: Don't go through the binding manager to see if our parent is an SVG element.

https://reviewboard.mozilla.org/r/233022/#review238538

r=me if you land this on top of my proposed changes in bug 1450617.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2a01fe0d3686
Don't go through the binding manager to see if our parent is an SVG element. r=bz

Comment 5

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/2a01fe0d3686