33162 2010-01-04 11:31:11 -0800 Releasing an initialized WebView without calling IWebView::close or DestroyWindow leaks the WebView 2011-09-26 05:16:33 -0700 1 1 1 Unclassified WebKit WebKit API 528+ (Nightly build) PC Windows XP REOPENED InRadar, PlatformOnly P2 Normal --- 1 aroben webkit-unassigned bfulgham rls.webkit oldest_to_newest 176771 0 aroben 2010-01-04 11:31:11 -0800 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. 176772 1 aroben 2010-01-04 11:31:38 -0800 <rdar://problem/7507492> 192474 2 aroben 2010-02-22 08:46:08 -0800 I think this might be fixed. 192493 3 aroben 2010-02-22 09:48:45 -0800 Yes, this was fixed somewhere in r52829-52830. 208534 4 rls.webkit 2010-04-05 10:16:56 -0700 (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? 208549 5 aroben 2010-04-05 10:33:21 -0700 Sure, I guess reopening this is a good way to handle it. 472250 6 108478 aroben 2011-09-23 08:20:32 -0700 Created attachment 108478 Remove two failing WebKitAPITest tests 473061 7 108478 aroben 2011-09-26 05:16:13 -0700 Comment on attachment 108478 Remove two failing WebKitAPITest tests Landed in r95943. 108478 2011-09-23 08:20:32 -0700 2011-09-26 05:16:33 -0700 Remove two failing WebKitAPITest tests bug-33162-20110923112040.patch text/plain 2379 aroben U3VidmVyc2lvbiBSZXZpc2lvbjogOTU3MzUKZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBi L1Rvb2xzL0NoYW5nZUxvZwppbmRleCA2M2UwNzI3MzQzMTJiMzVjZGQyMTIzMzA5MGMwYjcxYTk4 NzY2NTkwLi4xODY1YjdmYjA0Y2JiNDBkYjk5OTc0YjBjZjc5Mjk5MGI1M2RjODgzIDEwMDY0NAot LS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQg QEAKKzIwMTEtMDktMjMgIEFkYW0gUm9iZW4gIDxhcm9iZW5AYXBwbGUuY29tPgorCisgICAgICAg IFJlbW92ZSB0d28gZmFpbGluZyBXZWJLaXRBUElUZXN0IHRlc3RzCisKKyAgICAgICAgVGhlIGZh aWx1cmVzIGFyZSBjb3ZlcmVkIGJ5IDxodHRwOi8vd2Via2l0Lm9yZy9iLzMzMTYyPiBhbmQgPGh0 dHA6Ly93ZWJraXQub3JnL2IvMzI4Njg+LgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo T09QUyEpLgorCisgICAgICAgICogV2ViS2l0QVBJVGVzdC90ZXN0cy9XZWJWaWV3RGVzdHJ1Y3Rp b24uY3BwOiBSZW1vdmVkIHRoZSBOb0Nsb3NlT3JEZXN0cm95Vmlld1dpbmRvdyBhbmQKKyAgICAg ICAgTWFpbkZyYW1lQWZ0ZXJDbG9zZSB0ZXN0cy4KKwogMjAxMS0wOS0yMiAgQWRhbSBSb2JlbiAg PGFyb2JlbkBhcHBsZS5jb20+CiAKICAgICAgICAgUmVtb3ZlIEZpbmRTYWZhcmkKZGlmZiAtLWdp dCBhL1Rvb2xzL1dlYktpdEFQSVRlc3QvdGVzdHMvV2ViVmlld0Rlc3RydWN0aW9uLmNwcCBiL1Rv b2xzL1dlYktpdEFQSVRlc3QvdGVzdHMvV2ViVmlld0Rlc3RydWN0aW9uLmNwcAppbmRleCBiN2Fl MGI2YjgxNjBlYjE5YTZmMmQ5MTk1NWQwNmYzYTNlNmY4N2UwLi40YWIzYTZhMDczNDI0M2I5YTQw OWQxYmFlMTVlYjBiMTQ3YTAyYTA4IDEwMDY0NAotLS0gYS9Ub29scy9XZWJLaXRBUElUZXN0L3Rl c3RzL1dlYlZpZXdEZXN0cnVjdGlvbi5jcHAKKysrIGIvVG9vbHMvV2ViS2l0QVBJVGVzdC90ZXN0 cy9XZWJWaWV3RGVzdHJ1Y3Rpb24uY3BwCkBAIC0xMTQsMTcgKzExNCw2IEBAIFRFU1QoV2ViVmll d0Rlc3RydWN0aW9uLCBDbG9zZVdpdGhvdXRJbml0V2l0aEZyYW1lKQogICAgIGZpbmlzaFdlYlZp ZXdEZXN0cnVjdGlvblRlc3Qod2ViVmlldywgMCk7CiB9CiAKLS8vIFRlc3RzIHRoYXQgcmVsZWFz aW5nIGEgV2ViVmlldyB3aXRob3V0IGNhbGxpbmcgSVdlYlZpZXc6OmNsb3NlIG9yIERlc3Ryb3lX aW5kb3cgZG9lc24ndCBsZWFrLiA8aHR0cDovL3dlYmtpdC5vcmcvYi8zMzE2Mj4KLVRFU1QoV2Vi Vmlld0Rlc3RydWN0aW9uLCBOb0Nsb3NlT3JEZXN0cm95Vmlld1dpbmRvdykKLXsKLSAgICBDT01Q dHI8SVdlYlZpZXc+IHdlYlZpZXc7Ci0gICAgSG9zdFdpbmRvdyB3aW5kb3c7Ci0gICAgSFdORCB2 aWV3V2luZG93OwotICAgIGNyZWF0ZUFuZEluaXRpYWxpemVXZWJWaWV3KHdlYlZpZXcsIHdpbmRv dywgdmlld1dpbmRvdyk7Ci0KLSAgICBmaW5pc2hXZWJWaWV3RGVzdHJ1Y3Rpb25UZXN0KHdlYlZp ZXcsIHZpZXdXaW5kb3cpOwotfQotCiAvLyBUZXN0cyB0aGF0IGNhbGxpbmcgSVdlYlZpZXc6OmNs b3NlIHdpdGhvdXQgY2FsbGluZyBEZXN0cm95V2luZG93LCB0aGVuIHJlbGVhc2luZyBhIFdlYlZp ZXcgZG9lc24ndCBjcmFzaC4gPGh0dHA6Ly93ZWJraXQub3JnL2IvMzI4Mjc+CiBURVNUKFdlYlZp ZXdEZXN0cnVjdGlvbiwgQ2xvc2VXaXRob3V0RGVzdHJveVZpZXdXaW5kb3cpCiB7CkBAIC0yMTQs MTkgKzIwMyw0IEBAIFRFU1QoV2ViVmlld0Rlc3RydWN0aW9uLCBDbG9zZVRoZW5EZXN0cm95SG9z dFdpbmRvdykKICAgICBmaW5pc2hXZWJWaWV3RGVzdHJ1Y3Rpb25UZXN0KHdlYlZpZXcsIHZpZXdX aW5kb3cpOwogfQogCi0vLyBUZXN0cyB0aGF0IGNhbGxpbmcgSVdlYlZpZXc6Om1haW5GcmFtZSBh ZnRlciBjYWxsaW5nIElXZWJWaWV3OjpjbG9zZSBkb2Vzbid0IGNyYXNoLiA8aHR0cDovL3dlYmtp dC5vcmcvYi8zMjg2OD4KLVRFU1QoV2ViVmlld0Rlc3RydWN0aW9uLCBNYWluRnJhbWVBZnRlckNs b3NlKQotewotICAgIENPTVB0cjxJV2ViVmlldz4gd2ViVmlldzsKLSAgICBIb3N0V2luZG93IHdp bmRvdzsKLSAgICBIV05EIHZpZXdXaW5kb3c7Ci0gICAgY3JlYXRlQW5kSW5pdGlhbGl6ZVdlYlZp ZXcod2ViVmlldywgd2luZG93LCB2aWV3V2luZG93KTsKLQotICAgIFRFU1RfQVNTRVJUKFNVQ0NF RURFRCh3ZWJWaWV3LT5jbG9zZSgpKSk7Ci0gICAgQ09NUHRyPElXZWJGcmFtZT4gbWFpbkZyYW1l OwotICAgIFRFU1RfQVNTRVJUKFNVQ0NFRURFRCh3ZWJWaWV3LT5tYWluRnJhbWUoJm1haW5GcmFt ZSkpKTsKLQotICAgIGZpbmlzaFdlYlZpZXdEZXN0cnVjdGlvblRlc3Qod2ViVmlldywgdmlld1dp bmRvdyk7Ci19Ci0KIH0gLy8gbmFtZXNwYWNlIFdlYktpdEFQSVRlc3QK