Bug 20350 – [GTK] Get DumpRenderTree working

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Alp Toker]

DumpRenderTree is a valuable tool for automated testing. Attached patch completes basic functionality (without pixel dump or input event synthesis support).

Due to differences in fonts and metrics between Mac and GTK+ many of the render tree outputs don't match the existing ones in SVN. It's probably best to solve this (perhaps by loading custom fonts for DRT) before checking in any test results and enabling this on the build bot.

First impressions look good. About 50% of the tests pass, another 40% generate new reference output and the rest are failures or crashers (I'm fixing a few of these).

[Mark Rowe (bdash)]

A good first step may be to add a flag to run-webkit-tests that has it ignore non-text-only results (eg, render tree dumps).  Dealing with the font metric issue for render trees is rather tricky, and I think a lot more benefit can be gained from a) fixing the obvious crashers, and b) fixing any text-only tests that fail, before working on the render tree side of things.

[Alp Toker]

Created attachment 22835 [details]
DRT fixes

Fixes and enhancements to GTK+'s DRT tool.

[Holger Freyther]

Comment on attachment 22835 [details]
DRT fixes

Great step ahead.

You should get various new unused parameter warnings, e.g. just omit the parameter name and you are fine.

The flushing should fix quite some issues.

The cosmetic changes are I fine and besides personal taste I could not find anything in the Coding Style Guidelines to forbid comments at the end of the line.

The removal of WorkQueue::shared()->clear() and addition of webkit_web_view_open is something the other DRTs don't do. You are definitely right that some results depend on the order of the tests...

[Jan Alonzo]

landed in r35816