Bug 33162 – Releasing an initialized WebView without calling IWebView::close or DestroyWindow leaks the WebView

• Component: WebKit API
• Status: REOPENED

[Adam Roben (:aroben)]

If a WebKit application does the following:

IWebView* webView;
WebKitCreateInstance(CLSID_WebView, 0, IID_IWebView, reinterpret_cast<void**>(&webView));
webView->initWithFrame(rect, 0, 0);
webView->Release();

...then the WebView will be leaked.

[Adam Roben (:aroben)]

<rdar://problem/7507492>

[Adam Roben (:aroben)]

I think this might be fixed.

[Adam Roben (:aroben)]

Yes, this was fixed somewhere in r52829-52830.

[Russ Schnapp]

(In reply to comment #3)
> Yes, this was fixed somewhere in r52829-52830.

Actually, I'm not sure this is really fixed. In rev 57076, WebKitAPITest WebViewDestruction, NoCloseOrDestroyViewWindow is failing:

FAIL: WebViewDestruction: NoCloseOrDestroyViewWindow
        webViewCount() == currentWebViewCount - 1 (.\tests\WebViewDestruction.cpp:92)

After assigning 0 to webView, statistics->webViewCount() is still returning a value of 1.

I found this while running the Windows build. Should I reopen this bug?

[Adam Roben (:aroben)]

Sure, I guess reopening this is a good way to handle it.

[Adam Roben (:aroben)]

Created attachment 108478 [details]
Remove two failing WebKitAPITest tests

[Adam Roben (:aroben)]

Comment on attachment 108478 [details]
Remove two failing WebKitAPITest tests

Landed in r95943.