|Summary:||don't show missing text results for tests that only dump image results|
|Product:||WebKit||Reporter:||Ojan Vafai <ojan>|
|Component:||New Bugs||Assignee:||Ojan Vafai <ojan>|
|Severity:||Normal||CC:||dpranke, eric, tony|
|Version:||528+ (Nightly build)|
|OS:||OS X 10.5|
Description Ojan Vafai 2011-04-19 15:16:15 PDT
don't show missing text results for tests that only dump image results
Comment 2 Eric Seidel (no email) 2011-04-19 15:28:35 PDT
Comment on attachment 90263 [details] Patch I didn't know we had tests which only dumped images?
Comment 3 Dirk Pranke 2011-04-19 15:33:06 PDT
It is possible for any given test to be missing either text, audio, or images (but it should always produce either text or audio). It should not be possible for a test to only produce image data.
Comment 4 Ojan Vafai 2011-04-19 15:38:40 PDT
Oh, I'm wrong. It looks like if the text result passes and the image result is missing, we'll only dump the image result.
Comment 5 Dirk Pranke 2011-04-19 16:20:27 PDT
to further clarify, even though a test will always produce either text or audio, it will only save the output into the layout-test-results directory if the output didn't match the expected content. Ojan's patch is concerned with the saved output (the -actual.* files), not the produced output that lives in memory. Patch LGTM as well.