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=faabaae5574243508435fe3c...

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