[1/4] mshtml: Ignore IActiveScript interface for IXMLHTTPRequest (resend)
Alistair Leslie-Hughes
leslie_alistair at hotmail.com
Tue Feb 15 02:26:10 CST 2011
Hi Jacek,
On 14/02/2011 9:48 PM, Jacek Caban wrote:
> On 2/14/11 10:03 AM, Alistair Leslie-Hughes wrote:
>> Changelog:
>> mshtml: Ignore IActiveScript interface for IXMLHTTPRequest
>
> mshtml?
>
> + else if ( IsEqualGUID( riid,&IID_IActiveScript))
> + {
> + TRACE("(%p)->(IID_IActiveScript %p) returning NULL\n", This, ppvObject);
> + *ppvObject = NULL;
> + return E_NOINTERFACE;
> + }
> else
> {
> FIXME("Unsupported interface %s\n", debugstr_guid(riid));
>
> As discussed on wine-devel, please just change the later FIXME to TRACE
> instead.
By just changing the FIXME to a TRACE would hide this fact that
IObjectWithSite and IObjectSafety need to be implemented at some point.
Only with msxml tracing would be see this fact, which would lead to hard
to pin point errors.
We need to just ignore two know interfaces IActiveScript and IDispatchEx.
If we are going to stop silencing interfaces, Should be just remove the
ones already in webbrowser(shdocvw)? No, We do this so other developers
can see that these interfaces are not supported and so they don't need
to waste there time researching them. We silence interfaces like the
above in other places, so why no here?
Best Regards
Alistair Leslie-Hughes
More information about the wine-devel
mailing list