Re: [PATCH v3 0/5] MR8821: mshtml: Implement DOMParser.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/omnavigator.c:
> -
>    return (nsres == NS_ERROR_NOT_IMPLEMENTED) ? E_INVALIDARG : map_nsresult(nsres);
> -
> - if(strcmp(content_type, "text/html")) {
> -
>    nsAString_InitDepend(&errns, L"http://www.mozilla.org/newlayout/xml/parsererror.xml");
> -
>    nsAString_InitDepend(&errtag, L"parsererror");
> -
>    nsres = nsIDOMDocument_GetElementsByTagNameNS(nsdoc, &errns, &errtag, &nodes);
> -
>    nsAString_Finish(&errtag);
> -
>    nsAString_Finish(&errns);
> -
>    if(NS_SUCCEEDED(nsres)) {
> -
>        UINT32 length;
> -
>        nsres = nsIDOMNodeList_GetLength(nodes, &length);
> -
>        nsIDOMNodeList_Release(nodes);
> -
>        if(NS_SUCCEEDED(nsres) && length) {
> -
>            nsIDOMDocument_Release(nsdoc);
> -
>            return MSHTML_E_SYNTAX;
> -
>        }

Some debug trace of this happening would be nice.