Jacek Caban : mshtml: Added PluginHost::GetContainer implementation.

Alexandre Julliard julliard at winehq.org
Wed Dec 15 11:29:56 CST 2010


Module: wine
Branch: master
Commit: 94b591cdf226693de946971d76d7273951966fd8
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=94b591cdf226693de946971d76d7273951966fd8

Author: Jacek Caban <jacek at codeweavers.com>
Date:   Wed Dec 15 00:42:44 2010 +0100

mshtml: Added PluginHost::GetContainer implementation.

---

 dlls/mshtml/pluginhost.c    |   13 +++++++++++--
 dlls/mshtml/tests/activex.c |   27 +++++++++++++++++++++++++++
 2 files changed, 38 insertions(+), 2 deletions(-)

diff --git a/dlls/mshtml/pluginhost.c b/dlls/mshtml/pluginhost.c
index 9ab0983..f037eca 100644
--- a/dlls/mshtml/pluginhost.c
+++ b/dlls/mshtml/pluginhost.c
@@ -364,8 +364,17 @@ static HRESULT WINAPI PHClientSite_GetMoniker(IOleClientSite *iface, DWORD dwAss
 static HRESULT WINAPI PHClientSite_GetContainer(IOleClientSite *iface, IOleContainer **ppContainer)
 {
     PluginHost *This = impl_from_IOleClientSite(iface);
-    FIXME("(%p)->(%p)\n", This, ppContainer);
-    return E_NOTIMPL;
+
+    TRACE("(%p)->(%p)\n", This, ppContainer);
+
+    if(!This->doc) {
+        ERR("Called on detached object\n");
+        return E_UNEXPECTED;
+    }
+
+    *ppContainer = &This->doc->basedoc.IOleContainer_iface;
+    IOleContainer_AddRef(*ppContainer);
+    return S_OK;
 }
 
 static HRESULT WINAPI PHClientSite_ShowObject(IOleClientSite *iface)
diff --git a/dlls/mshtml/tests/activex.c b/dlls/mshtml/tests/activex.c
index a89d7e7..1036577 100644
--- a/dlls/mshtml/tests/activex.c
+++ b/dlls/mshtml/tests/activex.c
@@ -1164,6 +1164,32 @@ static const IClassFactoryVtbl ClassFactoryVtbl = {
 
 static IClassFactory activex_cf = { &ClassFactoryVtbl };
 
+static void test_container(IHTMLDocument2 *doc_obj)
+{
+    IHTMLWindow2 *parent_window;
+    IOleContainer *container;
+    IHTMLDocument2 *doc;
+    HRESULT hres;
+
+    container = NULL;
+    hres = IOleClientSite_GetContainer(client_site, &container);
+    ok(hres == S_OK, "GetContainer failed: %08x\n", hres);
+    ok(container != NULL, "container == NULL\n");
+
+    hres = IHTMLDocument2_get_parentWindow(doc_obj, &parent_window);
+    ok(hres == S_OK, "get_parentWindow failed: %08x\n", hres);
+    ok(parent_window != NULL, "parentWindow == NULL\n");
+
+    hres = IHTMLWindow2_get_document(parent_window, &doc);
+    ok(hres == S_OK, "get_document failed: %08x\n", hres);
+    ok(doc != NULL, "doc == NULL\n");
+    ok(iface_cmp((IUnknown*)doc, (IUnknown*)container), "container != doc\n");
+    IHTMLDocument2_Release(doc);
+
+    IHTMLWindow2_Release(parent_window);
+    IOleContainer_Release(container);
+}
+
 static void test_ui_activate(void)
 {
     IOleInPlaceSite *ip_site;
@@ -1779,6 +1805,7 @@ static void test_object_ax(void)
     CHECK_CALLED(SetObjectRects);
 
     test_ui_activate();
+    test_container(notif_doc);
 
     SET_EXPECT(UIDeactivate);
     SET_EXPECT(Invoke_ENABLED);




More information about the wine-cvs mailing list