Module: wine Branch: refs/heads/master Commit: faabaae5574243508435fe3c23192de8c46fec89 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=faabaae5574243508435fe3c... Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sun Apr 30 23:48:05 2006 +0200 shdocvw: Added some comments. --- dlls/shdocvw/webbrowser.c | 25 +++++++++++++++++++++++-- 1 files changed, 23 insertions(+), 2 deletions(-) diff --git a/dlls/shdocvw/webbrowser.c b/dlls/shdocvw/webbrowser.c index ff45c40..d50bbf3 100644 --- a/dlls/shdocvw/webbrowser.c +++ b/dlls/shdocvw/webbrowser.c @@ -317,6 +317,8 @@ static HRESULT WINAPI WebBrowser_put_Lef memcpy(&rect, &This->pos_rect, sizeof(RECT)); rect.left = Left; + /* We don't really change the window position here. + * We just notify the embedder that he should do so. */ return IOleInPlaceSite_OnPosRectChange(This->inplace, &rect); } @@ -343,6 +345,8 @@ static HRESULT WINAPI WebBrowser_put_Top memcpy(&rect, &This->pos_rect, sizeof(RECT)); rect.top = Top; + /* We don't really change the window position here. + * We just notify the embedder that he should do so. */ return IOleInPlaceSite_OnPosRectChange(This->inplace, &rect); } @@ -368,8 +372,10 @@ static HRESULT WINAPI WebBrowser_put_Wid memcpy(&rect, &This->pos_rect, sizeof(RECT)); rect.right = rect.left+Width; - - return IOleInPlaceSite_OnPosRectChange(This->inplace, &rect); + + /* We don't really change the window size here. + * We just notify the embedder that he should do so. */ + return IOleInPlaceSite_OnPosRectChange(This->inplace, &rect); } static HRESULT WINAPI WebBrowser_get_Height(IWebBrowser2 *iface, long *pl) @@ -395,6 +401,8 @@ static HRESULT WINAPI WebBrowser_put_Hei memcpy(&rect, &This->pos_rect, sizeof(RECT)); rect.bottom = rect.top+Height; + /* We don't really change the window size here. + * We just notify the embedder that he should do so. */ return IOleInPlaceSite_OnPosRectChange(This->inplace, &rect); } @@ -469,6 +477,7 @@ static HRESULT WINAPI WebBrowser_get_HWN TRACE("(%p)->(%p)\n", This, pHWND); + /* WebBrowser control never has a frame window (in opposition to InternetExplorer) */ *pHWND = 0; return E_FAIL; } @@ -534,6 +543,9 @@ static HRESULT WINAPI WebBrowser_put_Sta This->status_bar = Value; + /* In opposition to InternetExplorer, all we should do here is + * inform the embedder about the status bar change. */ + V_VT(&arg) = VT_BOOL; V_BOOL(&arg) = Value; call_sink(This->doc_host.cp_wbe2, DISPID_ONSTATUSBAR, &dispparams); @@ -575,6 +587,9 @@ static HRESULT WINAPI WebBrowser_put_Too This->tool_bar = Value ? VARIANT_TRUE : VARIANT_FALSE; + /* In opposition to InternetExplorer, all we should do here is + * inform the embedder about the tool bar change. */ + V_VT(&arg) = VT_BOOL; V_BOOL(&arg) = Value; call_sink(This->doc_host.cp_wbe2, DISPID_ONTOOLBAR, &dispparams); @@ -602,6 +617,9 @@ static HRESULT WINAPI WebBrowser_put_Men This->menu_bar = Value; + /* In opposition to InternetExplorer, all we should do here is + * inform the embedder about the menu bar change. */ + V_VT(&arg) = VT_BOOL; V_BOOL(&arg) = Value; call_sink(This->doc_host.cp_wbe2, DISPID_ONMENUBAR, &dispparams); @@ -799,6 +817,9 @@ static HRESULT WINAPI WebBrowser_put_Add This->address_bar = Value; + /* In opposition to InternetExplorer, all we should do here is + * inform the embedder about the address bar change. */ + V_VT(&arg) = VT_BOOL; V_BOOL(&arg) = Value; call_sink(This->doc_host.cp_wbe2, DISPID_ONADDRESSBAR, &dispparams);