[webkit-reviews] review denied: [Bug 30009] Web Inspector: close inspector client view on InspectorController::close API call. : [Attachment 40580] patch with comments addressed

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Oct 3 16:10:23 PDT 2009


Timothy Hatcher <timothy at hatcher.name> has denied Pavel Feldman
<pfeldman at chromium.org>'s request for review:
Bug 30009: Web Inspector: close inspector client view on
InspectorController::close API call.
https://bugs.webkit.org/show_bug.cgi?id=30009

Attachment 40580: patch with comments addressed
https://bugs.webkit.org/attachment.cgi?id=40580&action=review

------- Additional Comments from Timothy Hatcher <timothy at hatcher.name>
I think you will need:

    if (m_hwnd)
	::DestroyWindow(m_hwnd);


Also:

    if (inspectorView)
	delete inspectorView;


More information about the webkit-reviews mailing list