[webkit-reviews] review requested: [Bug 32827] Make it unnecessary for clients to call DestroyWindow on a WebView : [Attachment 45337] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Dec 21 10:15:36 PST 2009
Adam Roben (aroben) <aroben at apple.com> has asked for review:
Bug 32827: Make it unnecessary for clients to call DestroyWindow on a WebView
https://bugs.webkit.org/show_bug.cgi?id=32827
Attachment 45337: Patch
https://bugs.webkit.org/attachment.cgi?id=45337&action=review
------- Additional Comments from Adam Roben (aroben) <aroben at apple.com>
WebKit clients can currently cleanly get rid of a WebView in two ways.
A)
1) DestroyWindow(webViewHWND)
2) webView->Release()
B)
1) webView->close()
2) DestroyWindow(webViewHWND) (can be swapped with (1))
3) webView->Release()
We'd like clients to be able to get rid of a WebView just by releasing
the last reference to it. This patch gets us a little closer to that
by removing step B2 above (though calling DestroyWindow in this case
is harmless). A future patch will make steps A1 and B1 unnecessary, as
well.
Fixes <rdar://problem/7374218> Crash in WebView::updateActiveState
when closing "Welcome to iTunes" window
Reviewed by NOBODY (OOPS!).
* WebView.cpp:
(WebView::~WebView): Call setIsBeingDestroyed() so that we won't be
ref'd by our WndProc, which would result in this destructor being
re-entered.
(WebView::close): Moved the call to revokeDragDrop here...
(WebView::WebViewWndProc): ...from here. This is important in order to
release the reference that OLE holds while we're registered as a drop
target. Otherwise, clients that call IWebView::close but not
DestroyWindow would leak the WebView.
* WebView.h:
(WebView::setIsBeingDestroyed):
(WebView::isBeingDestroyed):
Made these private, and added a comment about what isBeingDestroyed()
now means.
---
3 files changed, 52 insertions(+), 5 deletions(-)
More information about the webkit-reviews
mailing list