Piotr Caban : mshtml: DeleteMemberByDispID should return E_NOTIMPL.

[Alexandre Julliard]

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

Author: Piotr Caban 
Date:   Tue Aug 23 11:33:10 2011 +0200

mshtml: DeleteMemberByDispID should return E_NOTIMPL.

---

dlls/mshtml/dispex.c       |    5 ++++-
 dlls/mshtml/tests/script.c |    3 +++
 2 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/dlls/mshtml/dispex.c b/dlls/mshtml/dispex.c
index 98f73b6..b1f9a12 100644
--- a/dlls/mshtml/dispex.c
+++ b/dlls/mshtml/dispex.c
@@ -1009,7 +1009,10 @@ static HRESULT WINAPI DispatchEx_DeleteMemberByName(IDispatchEx *iface, BSTR bst
 static HRESULT WINAPI DispatchEx_DeleteMemberByDispID(IDispatchEx *iface, DISPID id)
 {
     DispatchEx *This = impl_from_IDispatchEx(iface);
-    FIXME("(%p)->(%x)\n", This, id);
+
+    TRACE("(%p)->(%x)\n", This, id);
+
+    /* Not implemented by IE */
     return E_NOTIMPL;
 }

diff --git a/dlls/mshtml/tests/script.c b/dlls/mshtml/tests/script.c
index f0fd1dd..5cac14a 100644
--- a/dlls/mshtml/tests/script.c
+++ b/dlls/mshtml/tests/script.c
@@ -1951,6 +1951,9 @@ static void test_script_run(void)
     SysFreeString(tmp);
     ok(hres == S_OK, "GetDispID(document) failed: %08x\n", hres);

+    hres = IDispatchEx_DeleteMemberByDispID(document, id);
+    ok(hres == E_NOTIMPL, "DeleteMemberByDispID failed = %08x\n", hres);
+
     VariantInit(&var);
     memset(&dp, 0, sizeof(dp));
     memset(&ei, 0, sizeof(ei));