[PATCH v3 1/3] mshtml: Hold a ref to the location on the window side.
Gabriel Ivăncescu
gabrielopcode at gmail.com
Thu Nov 4 09:10:51 CDT 2021
Signed-off-by: Gabriel Ivăncescu <gabrielopcode at gmail.com>
---
dlls/mshtml/htmlwindow.c | 5 ++---
dlls/mshtml/tests/dom.c | 4 +---
2 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/dlls/mshtml/htmlwindow.c b/dlls/mshtml/htmlwindow.c
index 0b25e71..635b046 100644
--- a/dlls/mshtml/htmlwindow.c
+++ b/dlls/mshtml/htmlwindow.c
@@ -68,9 +68,7 @@ static inline BOOL is_outer_window(HTMLWindow *window)
static HRESULT get_location(HTMLInnerWindow *This, HTMLLocation **ret)
{
- if(This->location) {
- IHTMLLocation_AddRef(&This->location->IHTMLLocation_iface);
- }else {
+ if(!This->location) {
HRESULT hres;
hres = HTMLLocation_Create(This, &This->location);
@@ -78,6 +76,7 @@ static HRESULT get_location(HTMLInnerWindow *This, HTMLLocation **ret)
return hres;
}
+ IHTMLLocation_AddRef(&This->location->IHTMLLocation_iface);
*ret = This->location;
return S_OK;
}
diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c
index 03f5c04..ea5b210 100644
--- a/dlls/mshtml/tests/dom.c
+++ b/dlls/mshtml/tests/dom.c
@@ -6015,7 +6015,6 @@ static void test_location(IHTMLDocument2 *doc)
IHTMLLocation *location, *location2;
IHTMLWindow2 *window;
BSTR str;
- ULONG ref;
HRESULT hres;
hres = IHTMLDocument2_get_location(doc, &location);
@@ -6051,8 +6050,7 @@ static void test_location(IHTMLDocument2 *doc)
ok(!lstrcmpW(str, L"about:blank"), "unexpected href %s\n", wine_dbgstr_w(str));
SysFreeString(str);
- ref = IHTMLLocation_Release(location);
- ok(!ref, "location should be destroyed here\n");
+ IHTMLLocation_Release(location);
}
static void test_plugins_col(IHTMLDocument2 *doc)
--
2.31.1
More information about the wine-devel
mailing list