[webkit-reviews] review granted: [Bug 20350] [GTK] Get DumpRenderTree working : [Attachment 22835] DRT fixes

[bugzilla-daemon at webkit.org]

Holger Freyther <freyther at handhelds.org> has granted Alp Toker
<alp at nuanti.com>'s request for review:
Bug 20350: [GTK] Get DumpRenderTree working
https://bugs.webkit.org/show_bug.cgi?id=20350

Attachment 22835: DRT fixes
https://bugs.webkit.org/attachment.cgi?id=22835&action=edit

------- Additional Comments from Holger Freyther <freyther at handhelds.org>
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...