[Qt] Port DRT to use TestRunner::dumpAsText()
Created attachment 192923 [details] Patch
Comment on attachment 192923 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=192923&action=review > Tools/DumpRenderTree/qt/DumpRenderTreeQt.cpp:994 > - if (m_dumpPixelsForCurrentTest && m_controller->shouldDumpPixels()) { > + if (m_dumpPixelsForCurrentTest) { Shouldn't this use m_jscController->generatePixelResults() ? What do the other ports do there?
Created attachment 192928 [details] Patch
Comment on attachment 192928 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=192928&action=review Looks reasonable. > Tools/ChangeLog:15 > + TestRunnerQt::shouldDumpPixels() always returned true > + in our case and can therefore be replaced with > + TestRunner::generatePixelResults(). I am a little confused by this sentence.
Committed r145800: <http://trac.webkit.org/changeset/145800>