Jacek Caban : shdocvw: Added some comments.

[Alexandre Julliard]

Module: wine
Branch: refs/heads/master
Commit: faabaae5574243508435fe3c23192de8c46fec89
URL:    http://source.winehq.org/git/?p=wine.git;a=commit;h=faabaae5574243508435fe3c23192de8c46fec89

Author: Jacek Caban <jacek at 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);