Bug 31518 – [GTK] Support pixel tests

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

[Zan Dobersek]

This bug oversees the implementation of pixel dumping for the Gtk port.

[Zan Dobersek]

Created attachment 43250 [details]
Work-In-Progress of implementing pixel dumping support

This patch is work-in-progress and has some issues:
MD5 hashing should be reworked and tested to work on both Windows and Gtk port (we share this hashing part of the code with Windows port). This problem is better commented between the code. Also, MD5.h and MD5.cpp that Windows port possesses should be brought into the build rules for Gtk port. Currently, include of <openssl/md5.h>is used, but should be removed.
The diffing algorithm should be checked - it was rewritten from the Qt port and seems to work, but a check wouldn't harm.
The diff image creation should be looked upon - I didn't get onto any examples of what the diff image should actually look like, so I didn't finish that off completely.

More comments, suggestions etc. between the lines.

[Martin Robinson]

As I begin implementation of this patch, I'm splitting out different parts into different bugs.

[Martin Robinson]

Created attachment 60887 [details]
Convert PixelDumpSupportCairo.cpp to use NL line endings

[Xan Lopez]

Comment on attachment 60887 [details]
Convert PixelDumpSupportCairo.cpp to use NL line endings

r=me

[Martin Robinson]

Comment on attachment 60887 [details]
Convert PixelDumpSupportCairo.cpp to use NL line endings

Clearing flags on attachment: 60887

Committed r62839: <http://trac.webkit.org/changeset/62839>

[Martin Robinson]

Created attachment 60968 [details]
Pixel test support

[WebKit Review Bot]

Attachment 60968 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style', '--no-squash']" exit_code: 1
WebKitTools/DumpRenderTree/gtk/PixelDumpSupportGtk.cpp:32:  Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted.  [build/include_order] [4]
Total errors found: 1 in 7 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Martin Robinson]

Once again, I think this may be a false positive. This file is a platform-specific specialization implementation of an already platform-specific implementation. This means it doesn't have a header associated with it of the same name. Moving the PixelDumpSupportCairo.h include under config.h did not satisfy the style bot.

[Dirk Schulze]

Comment on attachment 60968 [details]
Pixel test support

r=me

[Martin Robinson]

Committed r63130: <http://trac.webkit.org/changeset/63130>