Bug 86145

Summary: [webkitpy][EFL] It should be possible to use a wrapper command for ImageDiff.
Product: WebKit Reporter: Raphael Kubo da Costa (:rakuco) <rakuco>
Component: Tools / TestsAssignee: Raphael Kubo da Costa (:rakuco) <rakuco>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dpranke, d-r, eric, ojan, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch dpranke: review+

Description Raphael Kubo da Costa (:rakuco) 2012-05-10 14:11:06 PDT
[webkitpy][EFL] It should be possible to use a wrapper command for ImageDiff.
Comment 1 Raphael Kubo da Costa (:rakuco) 2012-05-10 14:14:05 PDT
Created attachment 141249 [details]
Patch
Comment 2 Dirk Pranke 2012-05-10 14:19:57 PDT
Comment on attachment 141249 [details]
Patch

options are reserved for things that are actually command line flags. Please use a property or method on the object instead.
Comment 3 Raphael Kubo da Costa (:rakuco) 2012-05-10 14:37:37 PDT
Created attachment 141260 [details]
Patch
Comment 4 Raphael Kubo da Costa (:rakuco) 2012-05-10 14:59:42 PDT
Committed r116692: <http://trac.webkit.org/changeset/116692>