Commit 254bb55c authored by commit-queue@webkit.org's avatar commit-queue@webkit.org
Browse files

[Qt][WK2] Memory leaks in Qt's TestRunner

https://bugs.webkit.org/show_bug.cgi?id=96603

Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-09-13
Reviewed by Csaba Osztrogonác.

Fix memory leaks in Qt's TestRunner code by adopting WKString
created with WKStringCreateWithUTF8CString().

* WebKitTestRunner/qt/TestControllerQt.cpp:
(WTR::TestController::initializeInjectedBundlePath):
(WTR::TestController::initializeTestPluginDirectory):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128419 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 41ab4de1
2012-09-13 Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com>
[Qt][WK2] Memory leaks in Qt's TestRunner
https://bugs.webkit.org/show_bug.cgi?id=96603
Reviewed by Csaba Osztrogonác.
Fix memory leaks in Qt's TestRunner code by adopting WKString
created with WKStringCreateWithUTF8CString().
* WebKitTestRunner/qt/TestControllerQt.cpp:
(WTR::TestController::initializeInjectedBundlePath):
(WTR::TestController::initializeTestPluginDirectory):
2012-09-13 Zan Dobersek <zandobersek@gmail.com>
 
[GTK][NRWT] Lower timeout value to 6 seconds (12 for debug configuration)
......
......@@ -99,7 +99,7 @@ void TestController::initializeInjectedBundlePath()
if (!isExistingLibrary(path))
qFatal("Cannot find the injected bundle at %s\n", qPrintable(path));
m_injectedBundlePath = WKStringCreateWithQString(path);
m_injectedBundlePath.adopt(WKStringCreateWithQString(path));
}
void TestController::initializeTestPluginDirectory()
......@@ -107,7 +107,7 @@ void TestController::initializeTestPluginDirectory()
// FIXME: the test plugin cause some trouble for us, so we don't load it for the time being.
// See: https://bugs.webkit.org/show_bug.cgi?id=86620
// m_testPluginDirectory = WKStringCreateWithUTF8CString(qgetenv("QTWEBKIT_PLUGIN_PATH").constData());
// m_testPluginDirectory.adopt(WKStringCreateWithUTF8CString(qgetenv("QTWEBKIT_PLUGIN_PATH").constData()));
}
void TestController::platformInitializeContext()
......
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