[Qt][WK2]REGRESSION(r93784): It made many tests crash
authoralexis.menard@openbossa.org <alexis.menard@openbossa.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Aug 2011 18:24:48 +0000 (18:24 +0000)
committeralexis.menard@openbossa.org <alexis.menard@openbossa.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Aug 2011 18:24:48 +0000 (18:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=66958

Reviewed by Anders Carlsson.

Don't try to make sure the WebContext is deleted, for now the leak
is intended and when multiple processes per context will be supported
this leak should be fixed. This was the cause of DRT crashing on the bot,
the context was deleted after the first test, letting other tests with nothing
but a bad context.

* UIProcess/qt/QtWebPageProxy.cpp:
(QtWebPageProxy::~QtWebPageProxy):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@93897 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/qt/QtWebPageProxy.cpp

index d22e24f..d5a620c 100644 (file)
@@ -1,3 +1,19 @@
+2011-08-26  Alexis Menard  <alexis.menard@openbossa.org>
+
+        [Qt][WK2]REGRESSION(r93784): It made many tests crash
+        https://bugs.webkit.org/show_bug.cgi?id=66958
+
+        Reviewed by Anders Carlsson.
+
+        Don't try to make sure the WebContext is deleted, for now the leak
+        is intended and when multiple processes per context will be supported
+        this leak should be fixed. This was the cause of DRT crashing on the bot,
+        the context was deleted after the first test, letting other tests with nothing
+        but a bad context.
+
+        * UIProcess/qt/QtWebPageProxy.cpp:
+        (QtWebPageProxy::~QtWebPageProxy):
+
 2011-08-26  Anders Carlsson  <andersca@apple.com>
 
         Fix handling of keyup events in the new Cocoa text input model
index d22c3b0..7141e12 100644 (file)
@@ -210,7 +210,6 @@ void QtWebPageProxy::init()
 QtWebPageProxy::~QtWebPageProxy()
 {
     m_webPageProxy->close();
-    m_context->disconnectProcess(m_context->process());
     // The context is the default one and we're deleting the last QtWebPageProxy.
     if (m_context == s_defaultContext) {
         ASSERT(s_defaultPageProxyCount > 0);