Bug 21483 - Regression: run-webkit-tests --pixel svg fails with diffs which aren't diffs
Summary: Regression: run-webkit-tests --pixel svg fails with diffs which aren't diffs
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords: Regression
Depends on:
Blocks:
 
Reported: 2008-10-08 13:33 PDT by Eric Seidel (no email)
Modified: 2008-10-08 17:47 PDT (History)
1 user (show)

See Also:


Attachments
Patch, changelog (6.27 KB, patch)
2008-10-08 15:34 PDT, Simon Fraser (smfr)
eric: review+
Details | Formatted Diff | Diff
Alternate solution (6.53 KB, patch)
2008-10-08 17:41 PDT, Simon Fraser (smfr)
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2008-10-08 13:33:32 PDT
run-webkit-tests --pixel svg fails with diffs which aren't diffs

the perl script seems to get confused as to what's what.  It's spitting out png dumps as diffs and doing image diffs between the pixel results of the wrong two tests.  Something in the output stream from DRT must have gotten run-webkit-tests confused.

This is a regression as this used to work just fine.
Comment 1 Simon Fraser (smfr) 2008-10-08 13:38:41 PDT
Fallout from bug 21124 and bug 21323
Comment 2 Simon Fraser (smfr) 2008-10-08 15:34:19 PDT
Created attachment 24203 [details]
Patch, changelog

This patch changes DRT to print an ExpectPNG: line to the output when it knows it will spew out PNG data. The script can then look for this, rather than guessing. It also adds a little Content-Type checking.
Comment 3 Eric Seidel (no email) 2008-10-08 15:43:09 PDT
Comment on attachment 24203 [details]
Patch, changelog

Looks fine.
Comment 4 Simon Fraser (smfr) 2008-10-08 17:41:15 PDT
Created attachment 24210 [details]
Alternate solution

This second patch makes DRT always dump two #EOF-terminated blocks; the second, for PNG data, may be empty.
Comment 5 mitz 2008-10-08 17:43:53 PDT
Comment on attachment 24210 [details]
Alternate solution

r=me, assuming you tested in the various scenarios (--pixel, --pixel with no existing results, tests that dump as text and tests that don't etc.).
Comment 6 Simon Fraser (smfr) 2008-10-08 17:47:18 PDT
r37433 = 8c2bac03b93d3aa1c8b2667ebc9ce1c20dcb3d04 (trunk)
	M	WebKitTools/DumpRenderTree/cg/PixelDumpSupportCG.cpp
	M	WebKitTools/DumpRenderTree/win/DumpRenderTree.cpp
	M	WebKitTools/DumpRenderTree/mac/DumpRenderTree.mm
	M	WebKitTools/ChangeLog
	M	WebKitTools/Scripts/run-webkit-tests