Re: shdocvw 3: Added get_HWND implementation.

[Robert Shearman]

Jacek Caban wrote:
> static HRESULT WINAPI WebBrowser_get_HWND(IWebBrowser2 *iface, long *pHWND)
> {
>     WebBrowser *This = WEBBROWSER_THIS(iface);
> -    FIXME("(%p)->(%p)\n", This, pHWND);
> -    return E_NOTIMPL;
> +
> +    TRACE("(%p)->(%p)\n", This, pHWND);
> +
> +    *pHWND = 0;
> +    return E_FAIL;
> }

Again, this isn't implemented so you should keep the FIXME there.

-- 
Rob Shearman

shdocvw 3: Added get_HWND implementation.

[Jacek Caban]

Robert Shearman wrote:
> Jacek Caban wrote:
> > static HRESULT WINAPI WebBrowser_get_HWND(IWebBrowser2 *iface, long
> > *pHWND)
> > {
> >     WebBrowser *This = WEBBROWSER_THIS(iface);
> > -    FIXME("(%p)->(%p)\n", This, pHWND);
> > -    return E_NOTIMPL;
> > +
> > +    TRACE("(%p)->(%p)\n", This, pHWND);
> > +
> > +    *pHWND = 0;
> > +    return E_FAIL;
> > }
>
> Again, this isn't implemented so you should keep the FIXME there.

It's similar to put_Visible - this function is implemented only in
InternetExplorer. It should return frame window's HWND that doesn't
exist in control. I have test showing that it's all we have to do here.

Jacek