1016830 - Console.jsm fails when logging SVG elements

Status: RESOLVED WORKSFORME

(DevTools :: Console, defect)

Description

[Patrick Brosset <:pbro>]

0:06.53 A coding exception was thrown and uncaught in a Task.
 0:06.53 
 0:06.53 Full message: TypeError: aElement.className.split is not a function
 0:06.53 Full stack: debugElement@resource://gre/modules/devtools/Console.jsm:167:1
 0:06.53 stringify@resource://gre/modules/devtools/Console.jsm:130:7
 0:06.53 createDumper/</data<@resource://gre/modules/devtools/Console.jsm:471:7
 0:06.53 createDumper/<@resource://gre/modules/devtools/Console.jsm:470:9
 0:06.53 isHighlighterVisible@chrome://mochitests/content/browser/browser/devtools/markupview/test/head.js:228:21
 0:06.54 test<@chrome://mochitests/content/browser/browser/devtools/markupview/test/browser_markupview_highlight_hover_02.js:19:3
 0:06.54 TaskImpl_run@resource://gre/modules/Task.jsm:282:13
 0:06.54 Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:863:11
 0:06.54 this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:742:7

This happened when I tried to `console.log(element)` in a mochitest.
element, in this case, is a SVG node and apparently its className property is of type SVGAnimatedString which doesn't implement the split function (https://developer.mozilla.org/en-US/docs/Web/API/SVGAnimatedString).

Comment 1

[Robert Longson [:longsonr]]

Attachment: patch untested

Comment 2

[Robert Longson [:longsonr]]

How about this Patrick?

Comment 3

[Patrick Brosset <:pbro>]

Fails with the following error: "ReferenceError: SVGAnimatedString is not defined"
Would aElement.ownerDocument.defaultView.SVGAnimatedString work ?

Comment 4

[Robert Longson [:longsonr]]

Attachment: patch tested

This seems to work.

Comment 5

[Patrick Brosset <:pbro>]

This got fixed since I filed it. Apparently by bug 920141.