[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):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@93897 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 996a6b99
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
......
......@@ -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);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment