[PATCH v2 1/3] mshtml: Don't mess with the outer window if we're already detached.

[Gabriel Ivăncescu]

From: Gabriel Ivăncescu

Since now we always keep outer_window pointer valid (if it exists), we
could e.g. unlink location props from an already detached inner window. This
restores previous behavior on detach.

Fixes a regression introduced by 7c11ce8d44f1758a855c4d3c976825f8afe5fbb2.

Signed-off-by: Gabriel Ivăncescu 
---
 dlls/mshtml/htmlwindow.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/mshtml/htmlwindow.c b/dlls/mshtml/htmlwindow.c
index e685da74c03..ffcaf86a361 100644
--- a/dlls/mshtml/htmlwindow.c
+++ b/dlls/mshtml/htmlwindow.c
@@ -108,7 +108,7 @@ static inline HRESULT get_window_event(HTMLWindow *window, eventid_t eid, VARIAN

static void detach_inner_window(HTMLInnerWindow *window)
 {
-    HTMLOuterWindow *outer_window = window->base.outer_window;
+    HTMLOuterWindow *outer_window = is_detached_window(window) ? NULL : window->base.outer_window;
     HTMLDocumentNode *doc = window->doc, *doc_iter;

while(!list_empty(&window->children)) {