Re: [PATCH v2 0/1] MR3391: mshtml: implement window.MutationObserver with MutationObserver stub

[Jactry Zeng (＠jactry)]

Jactry Zeng (@jactry) commented about dlls/mshtml/mutation.c:
> +static inline struct mutation_observer *impl_from_IWineMSHTMLMutationObserver(IWineMSHTMLMutationObserver *iface)
> +{
> - return CONTAINING_RECORD(iface, struct mutation_observer, IWineMSHTMLMutationObserver_iface);
>
> +}
> -
>
> +static HRESULT WINAPI MutationObserver_QueryInterface(IWineMSHTMLMutationObserver *iface, REFIID riid, void **ppv)
> +{
> - struct mutation_observer *This = impl_from_IWineMSHTMLMutationObserver(iface);
> -
> - if(IsEqualGUID(&IID_IUnknown, riid) || IsEqualGUID(&IID_IWineMSHTMLMutationObserver, riid)) {
> -
>    *ppv = &This->IWineMSHTMLMutationObserver_iface;
> - }else {
> -
>    *ppv = NULL;
> -
>    return E_NOINTERFACE;
> - }
> -

I think adding a trace in QI() will be useful when debugging?