MSHTML: Active state handling
Jacek Caban
jack at itma.pwr.wroc.pl
Wed Jul 13 14:20:26 CDT 2005
Changelog:
- Better handling of UI and InPlace active state
- Implemented InPlaceDeactivate
- THIS macros cleanup
-------------- next part --------------
Index: dlls/mshtml/htmldoc.c
===================================================================
RCS file: /home/wine/wine/dlls/mshtml/htmldoc.c,v
retrieving revision 1.11
diff -u -p -r1.11 htmldoc.c
--- dlls/mshtml/htmldoc.c 12 Jul 2005 17:00:58 -0000 1.11
+++ dlls/mshtml/htmldoc.c 13 Jul 2005 19:12:56 -0000
@@ -128,13 +128,13 @@ static ULONG WINAPI HTMLDocument_Release
if(!ref) {
if(This->client)
- IOleClientSite_Release(This->client);
- if(This->hostui)
- IDocHostUIHandler_Release(This->hostui);
+ IOleObject_SetClientSite(OLEOBJ(This), NULL);
+ if(This->in_place_active)
+ IOleInPlaceObjectWindowless_InPlaceDeactivate(INPLACEWIN(This));
if(This->ipsite)
- IOleInPlaceSite_Release(This->ipsite);
- if(This->frame)
- IOleInPlaceFrame_Release(This->frame);
+ IOleDocumentView_SetInPlaceSite(DOCVIEW(This), NULL);
+ if(This->hwnd)
+ DestroyWindow(This->hwnd);
HeapFree(GetProcessHeap(), 0, This);
}
Index: dlls/mshtml/mshtml_private.h
===================================================================
RCS file: /home/wine/wine/dlls/mshtml/mshtml_private.h,v
retrieving revision 1.14
diff -u -p -r1.14 mshtml_private.h
--- dlls/mshtml/mshtml_private.h 13 Jul 2005 11:29:53 -0000 1.14
+++ dlls/mshtml/mshtml_private.h 13 Jul 2005 19:12:56 -0000
@@ -38,6 +38,9 @@ typedef struct {
IOleInPlaceFrame *frame;
HWND hwnd;
+
+ BOOL in_place_active;
+ BOOL ui_active;
} HTMLDocument;
#define HTMLDOC(x) ((IHTMLDocument2*) &(x)->lpHTMLDocument2Vtbl)
Index: dlls/mshtml/olewnd.c
===================================================================
RCS file: /home/wine/wine/dlls/mshtml/olewnd.c,v
retrieving revision 1.3
diff -u -p -r1.3 olewnd.c
--- dlls/mshtml/olewnd.c 13 Jul 2005 11:29:53 -0000 1.3
+++ dlls/mshtml/olewnd.c 13 Jul 2005 19:12:56 -0000
@@ -71,6 +71,9 @@ static HRESULT WINAPI OleInPlaceActiveOb
if(!phwnd)
return E_INVALIDARG;
+ if(!This->in_place_active)
+ return E_FAIL;
+
*phwnd = This->hwnd;
return S_OK;
}
@@ -175,8 +178,25 @@ static HRESULT WINAPI OleInPlaceObjectWi
static HRESULT WINAPI OleInPlaceObjectWindowless_InPlaceDeactivate(IOleInPlaceObjectWindowless *iface)
{
HTMLDocument *This = OLEINPLACEWND_THIS(iface);
- FIXME("(%p)\n", This);
- return E_NOTIMPL;
+
+ TRACE("(%p)\n", This);
+
+ if(!This->in_place_active)
+ return S_OK;
+
+ if(This->frame)
+ IOleInPlaceFrame_Release(This->frame);
+
+ if(This->hwnd) {
+ ShowWindow(This->hwnd, SW_HIDE);
+ SetWindowPos(This->hwnd, HWND_TOPMOST, 0, 0, 0, 0, SWP_NOSIZE | SWP_NOMOVE | SWP_NOACTIVATE);
+ }
+
+ This->in_place_active = FALSE;
+ if(This->ipsite)
+ IOleInPlaceSite_OnInPlaceDeactivate(This->ipsite);
+
+ return S_OK;
}
static HRESULT WINAPI OleInPlaceObjectWindowless_UIDeactivate(IOleInPlaceObjectWindowless *iface)
Index: dlls/mshtml/view.c
===================================================================
RCS file: /home/wine/wine/dlls/mshtml/view.c,v
retrieving revision 1.8
diff -u -p -r1.8 view.c
--- dlls/mshtml/view.c 13 Jul 2005 11:29:38 -0000 1.8
+++ dlls/mshtml/view.c 13 Jul 2005 19:12:57 -0000
@@ -96,35 +96,31 @@ static void register_serverwnd_class(voi
* IOleDocumentView implementation
*/
-#define DOCVIEW_THIS \
- HTMLDocument* const This=(HTMLDocument*)((char*)(iface)-offsetof(HTMLDocument,lpOleDocumentViewVtbl));
+#define DOCVIEW_THIS(iface) DEFINE_THIS(HTMLDocument, OleDocumentView, iface)
static HRESULT WINAPI OleDocumentView_QueryInterface(IOleDocumentView *iface, REFIID riid, void **ppvObject)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
return IHTMLDocument2_QueryInterface(HTMLDOC(This), riid, ppvObject);
}
static ULONG WINAPI OleDocumentView_AddRef(IOleDocumentView *iface)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
return IHTMLDocument2_AddRef(HTMLDOC(This));
}
static ULONG WINAPI OleDocumentView_Release(IOleDocumentView *iface)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
return IHTMLDocument2_Release(HTMLDOC(This));
}
static HRESULT WINAPI OleDocumentView_SetInPlaceSite(IOleDocumentView *iface, IOleInPlaceSite *pIPSite)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
TRACE("(%p)->(%p)\n", This, pIPSite);
- if(!pIPSite)
- return E_INVALIDARG;
-
if(pIPSite)
IOleInPlaceSite_AddRef(pIPSite);
@@ -137,7 +133,7 @@ static HRESULT WINAPI OleDocumentView_Se
static HRESULT WINAPI OleDocumentView_GetInPlaceSite(IOleDocumentView *iface, IOleInPlaceSite **ppIPSite)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
TRACE("(%p)->(%p)\n", This, ppIPSite);
if(!ppIPSite)
@@ -152,7 +148,7 @@ static HRESULT WINAPI OleDocumentView_Ge
static HRESULT WINAPI OleDocumentView_GetDocument(IOleDocumentView *iface, IUnknown **ppunk)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
TRACE("(%p)->(%p)\n", This, ppunk);
if(!ppunk)
@@ -165,7 +161,7 @@ static HRESULT WINAPI OleDocumentView_Ge
static HRESULT WINAPI OleDocumentView_SetRect(IOleDocumentView *iface, LPRECT prcView)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
RECT rect;
TRACE("(%p)->(%p)\n", This, prcView);
@@ -187,7 +183,7 @@ static HRESULT WINAPI OleDocumentView_Se
static HRESULT WINAPI OleDocumentView_GetRect(IOleDocumentView *iface, LPRECT prcView)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
TRACE("(%p)->(%p)\n", This, prcView);
@@ -201,14 +197,14 @@ static HRESULT WINAPI OleDocumentView_Ge
static HRESULT WINAPI OleDocumentView_SetRectComplex(IOleDocumentView *iface, LPRECT prcView,
LPRECT prcHScroll, LPRECT prcVScroll, LPRECT prcSizeBox)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
FIXME("(%p)->(%p %p %p %p)\n", This, prcView, prcHScroll, prcVScroll, prcSizeBox);
return E_NOTIMPL;
}
static HRESULT WINAPI OleDocumentView_Show(IOleDocumentView *iface, BOOL fShow)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
TRACE("(%p)->(%x)\n", This, fShow);
if(This->hwnd)
@@ -219,7 +215,7 @@ static HRESULT WINAPI OleDocumentView_Sh
static HRESULT WINAPI OleDocumentView_UIActivate(IOleDocumentView *iface, BOOL fUIActivate)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
HRESULT hres;
IOleInPlaceUIWindow *pIPWnd;
IOleInPlaceFrame *pIPFrame;
@@ -235,7 +231,7 @@ static HRESULT WINAPI OleDocumentView_UI
}
if(fUIActivate) {
- if(This->hwnd)
+ if(This->ui_active)
return S_OK;
if(!serverwnd_class)
register_serverwnd_class();
@@ -271,9 +267,11 @@ static HRESULT WINAPI OleDocumentView_UI
posrect.left, posrect.top, posrect.right-posrect.left, posrect.bottom-posrect.top,
parent_hwnd, NULL, hInst, This);
+ This->in_place_active = TRUE;
hres = IOleInPlaceSite_OnInPlaceActivate(This->ipsite);
if(FAILED(hres)) {
WARN("OnInPlaceActivate failed: %08lx\n", hres);
+ This->in_place_active = FALSE;
return hres;
}
@@ -288,11 +286,13 @@ static HRESULT WINAPI OleDocumentView_UI
* SetTimer(This->hwnd, TIMER_ID, 100, NULL);
*/
+ This->ui_active = TRUE;
hres = IOleInPlaceSite_OnUIActivate(This->ipsite);
if(SUCCEEDED(hres)) {
IOleInPlaceFrame_SetActiveObject(pIPFrame, ACTOBJ(This), wszHTML_Document);
}else {
FIXME("OnUIActivate failed: %08lx\n", hres);
+ This->ui_active = FALSE;
DestroyWindow(hwnd);
return hres;
}
@@ -306,6 +306,7 @@ static HRESULT WINAPI OleDocumentView_UI
if(FAILED(hres))
IDocHostUIHandler_HideUI(This->hostui);
}else {
+ This->ui_active = FALSE;
if(This->frame)
IOleInPlaceFrame_SetActiveObject(This->frame, NULL, NULL);
if(This->hostui)
@@ -318,14 +319,14 @@ static HRESULT WINAPI OleDocumentView_UI
static HRESULT WINAPI OleDocumentView_Open(IOleDocumentView *iface)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
FIXME("(%p)\n", This);
return E_NOTIMPL;
}
static HRESULT WINAPI OleDocumentView_CloseView(IOleDocumentView *iface, DWORD dwReserved)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
TRACE("(%p)->(%lx)\n", This, dwReserved);
if(dwReserved)
@@ -343,14 +344,14 @@ static HRESULT WINAPI OleDocumentView_Cl
static HRESULT WINAPI OleDocumentView_SaveViewState(IOleDocumentView *iface, LPSTREAM pstm)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
FIXME("(%p)->(%p)\n", This, pstm);
return E_NOTIMPL;
}
static HRESULT WINAPI OleDocumentView_ApplyViewState(IOleDocumentView *iface, LPSTREAM pstm)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
FIXME("(%p)->(%p)\n", This, pstm);
return E_NOTIMPL;
}
@@ -358,11 +359,13 @@ static HRESULT WINAPI OleDocumentView_Ap
static HRESULT WINAPI OleDocumentView_Clone(IOleDocumentView *iface, IOleInPlaceSite *pIPSiteNew,
IOleDocumentView **ppViewNew)
{
- DOCVIEW_THIS
+ HTMLDocument *This = DOCVIEW_THIS(iface);
FIXME("(%p)->(%p %p)\n", This, pIPSiteNew, ppViewNew);
return E_NOTIMPL;
}
+#undef DOCVIEW_THIS
+
static const IOleDocumentViewVtbl OleDocumentViewVtbl = {
OleDocumentView_QueryInterface,
OleDocumentView_AddRef,
@@ -386,24 +389,23 @@ static const IOleDocumentViewVtbl OleDoc
* IViewObject implementation
*/
-#define VIEWOBJ_THIS \
- HTMLDocument* const This=(HTMLDocument*)((char*)(iface)-offsetof(HTMLDocument,lpViewObject2Vtbl));
+#define VIEWOBJ_THIS(iface) DEFINE_THIS(HTMLDocument, ViewObject2, iface)
static HRESULT WINAPI ViewObject_QueryInterface(IViewObject2 *iface, REFIID riid, void **ppvObject)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
return IHTMLDocument2_QueryInterface(HTMLDOC(This), riid, ppvObject);
}
static ULONG WINAPI ViewObject_AddRef(IViewObject2 *iface)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
return IHTMLDocument2_AddRef(HTMLDOC(This));
}
static ULONG WINAPI ViewObject_Release(IViewObject2 *iface)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
return IHTMLDocument2_Release(HTMLDOC(This));
}
@@ -411,7 +413,7 @@ static HRESULT WINAPI ViewObject_Draw(IV
DVTARGETDEVICE *ptd, HDC hdcTargetDev, HDC hdcDraw, LPCRECTL lprcBounds,
LPCRECTL lprcWBounds, BOOL (CALLBACK *pfnContinue)(ULONG_PTR dwContinue), ULONG_PTR dwContinue)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
FIXME("(%p)->(%ld %ld %p %p %p %p %p %p %p %ld)\n", This, dwDrawAspect, lindex, pvAspect,
ptd, hdcTargetDev, hdcDraw, lprcBounds, lprcWBounds, pfnContinue, dwContinue);
return E_NOTIMPL;
@@ -420,7 +422,7 @@ static HRESULT WINAPI ViewObject_Draw(IV
static HRESULT WINAPI ViewObject_GetColorSet(IViewObject2 *iface, DWORD dwDrawAspect, LONG lindex, void *pvAspect,
DVTARGETDEVICE *ptd, HDC hicTargetDev, LOGPALETTE **ppColorSet)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
FIXME("(%p)->(%ld %ld %p %p %p %p)\n", This, dwDrawAspect, lindex, pvAspect, ptd, hicTargetDev, ppColorSet);
return E_NOTIMPL;
}
@@ -428,28 +430,28 @@ static HRESULT WINAPI ViewObject_GetColo
static HRESULT WINAPI ViewObject_Freeze(IViewObject2 *iface, DWORD dwDrawAspect, LONG lindex,
void *pvAspect, DWORD *pdwFreeze)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
FIXME("(%p)->(%ld %ld %p %p)\n", This, dwDrawAspect, lindex, pvAspect, pdwFreeze);
return E_NOTIMPL;
}
static HRESULT WINAPI ViewObject_Unfreeze(IViewObject2 *iface, DWORD dwFreeze)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
FIXME("(%p)->(%ld)\n", This, dwFreeze);
return E_NOTIMPL;
}
static HRESULT WINAPI ViewObject_SetAdvise(IViewObject2 *iface, DWORD aspects, DWORD advf, IAdviseSink *pAdvSink)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
FIXME("(%p)->(%ld %ld %p)\n", This, aspects, advf, pAdvSink);
return E_NOTIMPL;
}
static HRESULT WINAPI ViewObject_GetAdvise(IViewObject2 *iface, DWORD *pAspects, DWORD *pAdvf, IAdviseSink **ppAdvSink)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
FIXME("(%p)->(%p %p %p)\n", This, pAspects, pAdvf, ppAdvSink);
return E_NOTIMPL;
}
@@ -457,11 +459,13 @@ static HRESULT WINAPI ViewObject_GetAdvi
static HRESULT WINAPI ViewObject_GetExtent(IViewObject2 *iface, DWORD dwDrawAspect, LONG lindex,
DVTARGETDEVICE* ptd, LPSIZEL lpsizel)
{
- VIEWOBJ_THIS
+ HTMLDocument *This = VIEWOBJ_THIS(iface);
FIXME("(%p)->(%ld %ld %p %p)\n", This, dwDrawAspect, lindex, ptd, lpsizel);
return E_NOTIMPL;
}
+#undef VIEWOBJ_THIS
+
static const IViewObject2Vtbl ViewObjectVtbl = {
ViewObject_QueryInterface,
ViewObject_AddRef,
@@ -483,4 +487,7 @@ void HTMLDocument_View_Init(HTMLDocument
This->ipsite = NULL;
This->frame = NULL;
This->hwnd = NULL;
+
+ This->in_place_active = FALSE;
+ This->ui_active = FALSE;
}
More information about the wine-patches
mailing list