Bug 22209

Summary: Layout tests crash the Qt Browser
Product: WebKit Reporter: Krishna <krishnamurty.podipireddy>
Component: WebKit QtAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Critical CC: thisisbradley
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch Qt's DRT to send an empty block for the pixel data hausmann: review+

Description Krishna 2008-11-12 11:16:19 PST
I have been trying to run the Layout tests on my Ubuntu Linux desktop with QtWebkit. All the tests are crashing. For example when I execute the command 
run-webkit-tests LayoutTests/fast/forms/plaintext-mode-1.html, the test plaintext-mode-1.html is crashing.

Here is my environmetal variables:
LIBRARY_PATH=/usr/local/Trolltech/Qt-4.4.3/lib
QTDIR=/usr/local/Trolltech/Qt-4.4.3
LD_LIBRARY_PATH=/usr/local/Trolltech/Qt-4.4.3/lib
WEBKITDIR=/home/mobuntu/src/cwrt/Webkit
WEBKIT_TESTFONTS=/home/mobuntu/src/cwrt/Webkit/TestFonts

Also the results from plaintext-mode-1-stderr.txt file has got the following message though I have downloaded the test fonts and have set the environmental variable as shown above:
--------------------------------------------------------------------
WEBKIT_TESTFONTS environment variable is not set correctly.
This variable has to point to the directory containing the fonts
you can checkout from svn://labs.trolltech.com/svn/webkit/testfonts
--------------------------------------------------------------------

External svn revision: 38301
Internal svn revision: 8115
Comment 1 thisisbradley 2008-11-13 13:49:15 PST
run-layout-tests hung for me. Turns out as of http://trac.webkit.org/changeset/37434, run-layout-tests expects DRT to send two data blocks - one for the text output, one for the pixel data. Patch forthcoming. 
Comment 2 thisisbradley 2008-11-13 13:51:52 PST
Created attachment 25133 [details]
Patch Qt's DRT to send an empty block for the pixel data
Comment 3 thisisbradley 2008-11-13 13:53:10 PST
Comment on attachment 25133 [details]
Patch Qt's DRT to send an empty block for the pixel data

Forgot to set review flag
Comment 4 Simon Hausmann 2008-11-14 10:41:23 PST
Comment on attachment 25133 [details]
Patch Qt's DRT to send an empty block for the pixel data

Good catch!
Comment 5 Simon Hausmann 2008-11-14 10:44:25 PST
Landed in r38397