mshtml: Move detach_document_node call to HTMLDocumentNode_unlink.

Signed-off-by: Jacek Caban <jacek@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

mshtml: Move detach_document_node call to HTMLDocumentNode_unlink.
d9da6e64 · Jacek Caban · Alexandre Julliard · 43ae349c · d9da6e64
Commit d9da6e64 authored Mar 28, 2019 by Jacek Caban Committed by Alexandre Julliard Mar 28, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

htmldoc.c dlls/mshtml/htmldoc.c +3 -1

No files found.
--- a/dlls/mshtml/htmldoc.c
+++ b/dlls/mshtml/htmldoc.c
@@ -4966,7 +4966,8 @@ static void HTMLDocumentNode_destructor(HTMLDOMNode *iface)
 {
    HTMLDocumentNode *This = impl_from_HTMLDOMNode(iface);

-    detach_document_node(This);
+    TRACE("(%p)\n", This);
+
    heap_free(This->event_vector);
    ConnectionPointContainer_Destroy(&This->basedoc.cp_container);
 }
@@ -4994,6 +4995,7 @@ static void HTMLDocumentNode_unlink(HTMLDOMNode *iface)
        nsIDOMHTMLDocument *nsdoc = This->nsdoc;

        release_document_mutation(This);
+        detach_document_node(This);
        This->nsdoc = NULL;
        nsIDOMHTMLDocument_Release(nsdoc);
        This->window = NULL;