mshtml: Add IHTMLXMLHttpRequest::responseXML() method implementation. (try 2)
Zhenbo Li
litimetal at gmail.com
Sat Nov 14 03:33:28 CST 2015
Hi Jacek,
2015-11-13 19:39 GMT+08:00 Jacek Caban <jacek at codeweavers.com>:
>
>
> Sorry for the delay, I was waiting for wine-devel discussion to come to
> conclusion, but failed to reply when it did. It would be nice to move it
> from pending state, although I'm not sure we can do that before code freeze.
>
> +
> + hres = IXMLDOMDocument_loadXML(xmldoc, str, &vbool);
> + SysFreeString(str);
> + if(hres != S_OK || vbool != VARIANT_TRUE) {
> + ERR("loadXML failed: %08x\n", hres);
> + IXMLDOMDocument_Release(xmldoc);
> + return hres;
> + }
>
>
> Note that if loadXML fails, you will return S_FALSE here, which I'm sure is desired. It would be nice to have a test for that.
>
Yes, but I'm wondering how to test it, as we can find test cases for
IXMLDOMDocument_loadXML in msxml3/tests
>
> Also discussion with Nikolay convinced me that assert() would be a better way to handle IObjectSafety errors.
>
> Thanks,
> Jacek
>
>
>
Also, I found that there was a hidden bug:
As we will test these urls for responseXML:
static const char xml_url[] = "http://test.winehq.org/tests/xmltest.xml";
static const char large_page_url[] =
"http://test.winehq.org/tests/data.php";
char large_page_url is not a valid xml file, but native mshtml.dll can
handle it properly. It shows we can't just past the responseText to
msxml3.dll. I'll check it more carefully it later.
--
Have a nice day!
Zhenbo Li
More information about the wine-devel
mailing list