For localization purpose, we need the parser to support general entities declared in the external DTD file. (XML spec says an entity is internal if the entity definition is an EntityValue; otherwise, it is an external one). Also, the parser needs to provide a callback hook so that the locale information could be used to locate and fetch the language-specific DTD.

Add URL to XUL localizability spec. Also assign QA contact.

Accepting bug and changing component to XML.

Catalin is working on this and from my talk with him yesterday, I think this won't make it in time for M5. So, I'm moving this bug to M6, assigning this to Catalin and putting myself on the cc list.

The status of it: 1. It takes absolute URL as the systemid only. 2. It need doctype declaration in the DTD file. This shouldn't be the case. 3. It does nou support co-existence of internal and external DTD subset yet. I'll file separate bugs on them.

It's in, but the code is #ifdef'ed. Reassigning to Nisheeth so he can final test & review and turn the flag on.

Accepting bug... I tried testing the following XML and, with EXTERNAL_ENTITY_SUPPORT enabled, nothing displays in viewer: <?xml version="1.0"> <test> this text should display. </test> --- Ccing cata so that he can also take a look at this.

Updating summary.

I thought the correct syntax was: <?xml version="1.0"?> (please note the final "?") Maybe that's the problem?

You are right about the syntax. But, an error message should show up in the viewer. Currently, nothing shows up.

Oops, my bad. It seems that this behavior was occurring earlier also... I'm in there testing more as I type...

My testing did not uncover new issues: everything I found already has bugs open or was reported by James Clark. Clearing status because basic XML does display fine with these changes enabled. Changing summary to reflect that only enablement remains. I'm setting the target milestone for the resolution of this bug to M8. We need to address James Clarks' concerns as well as resolve bugs 8251, 8254, and 8255 before we turn this on for everybody. Adding those bugs to the bug dependency list.

I just turned this on in the builds of all platforms. Thanks a lot to James for providing a patch to expat that adds this new feature. Many thanks to Tao for integrating James' changes into the mozilla codebase.

Verified with jul-07-99 build.