[PATCH v16 2/2] mshtml: add stubs for MutationObserver methods

[Yuxuan Shui]

From: Yuxuan Shui

---
 dlls/mshtml/mshtml_private_iface.idl |  7 +++++++
 dlls/mshtml/mutation.c               | 31 ++++++++++++++++++++++++++++
 dlls/mshtml/tests/documentmode.js    | 10 ++++++++-
 3 files changed, 47 insertions(+), 1 deletion(-)

diff --git a/dlls/mshtml/mshtml_private_iface.idl b/dlls/mshtml/mshtml_private_iface.idl
index 46bae638fa5..e4d97361ce1 100644
--- a/dlls/mshtml/mshtml_private_iface.idl
+++ b/dlls/mshtml/mshtml_private_iface.idl
@@ -21,6 +21,7 @@
 #include <mshtmdid.h>

import "ocidl.idl";
+import "mshtml.idl";

[
     version(1.0),
@@ -85,6 +86,12 @@ interface IWineMSHTMLConsole : IDispatch
 ]
 interface IWineMSHTMLMutationObserver : IDispatch
 {
+    [id(1)]
+    HRESULT disconnect();
+    [id(2)]
+    HRESULT observe([in] IHTMLDOMNode *target, [in] IDispatch *options);
+    [id(3)]
+    HRESULT takeRecords([retval, out] IDispatch **records);
 }

[
diff --git a/dlls/mshtml/mutation.c b/dlls/mshtml/mutation.c
index 136cb05f9df..c058a3642e5 100644
--- a/dlls/mshtml/mutation.c
+++ b/dlls/mshtml/mutation.c
@@ -1171,6 +1171,34 @@ static HRESULT WINAPI MutationObserver_Invoke(IWineMSHTMLMutationObserver *iface
             pDispParams, pVarResult, pExcepInfo, puArgErr);
 }

+static HRESULT WINAPI MutationObserver_disconnect(IWineMSHTMLMutationObserver *iface)
+{
+    struct mutation_observer *This = impl_from_IWineMSHTMLMutationObserver(iface);
+
+    FIXME("(%p), stub\n", This);
+
+    return S_OK;
+}
+
+static HRESULT WINAPI MutationObserver_observe(IWineMSHTMLMutationObserver *iface, IHTMLDOMNode *target,
+                                               IDispatch *options)
+{
+    struct mutation_observer *This = impl_from_IWineMSHTMLMutationObserver(iface);
+
+    FIXME("(%p)->(%p %p), stub\n", This, target, options);
+
+    return S_OK;
+}
+
+static HRESULT WINAPI MutationObserver_takeRecords(IWineMSHTMLMutationObserver *iface, IDispatch **ret)
+{
+    struct mutation_observer *This = impl_from_IWineMSHTMLMutationObserver(iface);
+
+    FIXME("(%p)->(%p), stub\n", This, ret);
+
+    return E_NOTIMPL;
+}
+
 static const IWineMSHTMLMutationObserverVtbl WineMSHTMLMutationObserverVtbl = {
     MutationObserver_QueryInterface,
     MutationObserver_AddRef,
@@ -1179,6 +1207,9 @@ static const IWineMSHTMLMutationObserverVtbl WineMSHTMLMutationObserverVtbl = {
     MutationObserver_GetTypeInfo,
     MutationObserver_GetIDsOfNames,
     MutationObserver_Invoke,
+    MutationObserver_disconnect,
+    MutationObserver_observe,
+    MutationObserver_takeRecords
 };

static const tid_t mutation_observer_iface_tids[] = {
diff --git a/dlls/mshtml/tests/documentmode.js b/dlls/mshtml/tests/documentmode.js
index 35921ca565f..9f46de12466 100644
--- a/dlls/mshtml/tests/documentmode.js
+++ b/dlls/mshtml/tests/documentmode.js
@@ -2909,7 +2909,15 @@ sync_test("MutationObserver", function() {
     } catch(e) {
         ok(false, "MutationObserver with extra args threw exception " + e.number);
     }
-})
+
+    var mutation_observer = new MutationObserver(function() {});
+    function test_exposed(prop) {
+        ok(prop in mutation_observer, prop + " not found in MutationObserver.");
+    }
+    test_exposed("observe");
+    test_exposed("disconnect");
+    test_exposed("takeRecords");
+});

async_test("postMessage", function() {
     var v = document.documentMode;