Module: wine
Branch: refs/heads/master
Commit: faabaae5574243508435fe3c23192de8c46fec89
URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=faabaae5574243508435fe3…
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);