1514669 - Prepare for m-c removal of nsISAXXMLReader in XMPP

Status: RESOLVED DUPLICATE of bug 1518552

(Chat Core :: XMPP, enhancement)

Description

[Henri Sivonen (:hsivonen)]

Bug 1447707 will remove nsISAXXMLReader from m-c.

https://searchfox.org/comm-central/source/chat/protocols/xmpp/xmpp-xml.jsm uses it.

Comment 1

[Florian Quèze [:florian]]

(In reply to Henri Sivonen (:hsivonen) from comment #0)
> Bug 1447707 will remove nsISAXXMLReader from m-c.
> 
> https://searchfox.org/comm-central/source/chat/protocols/xmpp/xmpp-xml.jsm
> uses it.

Is there another supported way to parse an incoming XML stream?

Comment 2

[Henri Sivonen (:hsivonen)]

We don't have any other JS-exposed way to parse XML in a SAX-like way that doesn't involve DOM nodes. An <iframe> and I *think* XHR let you see the DOM tree as it's being built, though. Parsing via XHR and removing the old DOM nodes once you've seen them might work.

One option is to move nsISAXXMLReader, etc., to c-c.

Comment 3

[Henri Sivonen (:hsivonen)]

From the m-c perspective, the removal is now ready to land, but I could wait until the second week of January or so before landing it, since it seems potentially impolite to break c-c at this time of year. Are you planning to import the code to c-c, and if so, should this bug and bug 1514666 be collapsed into one?

Comment 4

[Jorg K (CEST = GMT+2)]

The Sax parser has now been removed as announced for the second week of January. So should we fork it?

Comment 5

[Jorg K (CEST = GMT+2)]

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

Comment 6

[Florian Quèze [:florian]]

The SAX parser was forked to comm-central in bug 1518552; closing as a duplicate, but thanks for the initial heads up Henri.