Re: [PATCH v2 0/6] MR8960: mshtml: Implement XDomainRequest.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/xmlhttprequest.c:
> -
>    hres = IUri_GetSchemeName(This->window->base.outer_window->uri, &bstr);
> -
>    if(FAILED(hres))
> -
>        return hres;
> -
>    if(SysStringLen(bstr) != len || wcsnicmp(bstr, bstrUrl, len))
> -
>        hres = E_ACCESSDENIED;
> -
>    SysFreeString(bstr);
> -
>    if(FAILED(hres))
> -
>        return hres;
> - }
> -
> - V_VT(&vtrue) = VT_BOOL;
> - V_BOOL(&vtrue) = VARIANT_TRUE;
> - V_VT(&vempty) = VT_EMPTY;
> - magic = This->magic;
> - hres = HTMLXMLHttpRequest_open(&This->IHTMLXMLHttpRequest_iface, bstrMethod, bstrUrl, vtrue, vempty, vempty);
> - if(FAILED(hres) || magic + 1 != This->magic)

Why do we need to worry about magic here? Shouldn't we set response time anyway?