102084 - Synchronous document loading function needed

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P3)

Description

[Jon Smirl]

Is a synchronous document loading function needed with recursion detection to 
stop loops?

The following filles look to me like they are reimplementing the same thing, 
but I'm not an expert on the document loading sequence. Should nsSyncLoader 
should be moved out of Transformiix and into content/base? This object doesn't 
look like it belongs in the XSLT code and it seems to be reimplemented three 
times that I can find. 
 
These two files look like they are reimplementing synchronous XML document 
loading with recursion detection.
http://lxr.mozilla.org/seamonkey/source/extensions/xmlextras/base/src/nsDOMParse
r.cpp
http://lxr.mozilla.org/seamonkey/source/extensions/transformiix/source/xml/parse
r/nsSyncLoader.cpp
 
This one has some similar loading code but is more different.
http://lxr.mozilla.org/seamonkey/source/content/xbl/src/nsXBLService.cpp

Comment 1

[Johnny Stenback (:jst)]

Reassigning to peterv so that he can generalize the sync loading code in XSLT if
it makes sense, I'm not sure it does.

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

taking...

Comment 3

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Depending on how this gets implemented, this could be a dupe of bug 65985. If
this really can be implemented "the right way" this would fix a lot of bugs we
have with synchronous loading. If you have ideas how to do this, please present
them here. So far nobody has gotten farther than: it is impossible in embedded
applications. I'd like to be proven wrong :)

Comment 4

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

ugh, i seem to be missing quite a lot of bugmail, didn't see comment 3 until today.

Anyhow, there is now a nsISyncLoadDOMService that doesn't use a window to
syncload. Instead it pushes a new eventqueue and uses that to wait for the
document to load. (It's pretty much just a refactor of petervs code from
transformiix, so i can't take any credit for it).

Not sure if this will work in the embedding case, but i would think it should.

I'm marking this one fixed. Let's use bug 65985 for making xml-extras use the
new syncload service.