Summary: | PrettyPatch.rb complains about missing checksum for new pixel results | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tony Chang <tony> | ||||
Component: | Tools / Tests | Assignee: | Ojan Vafai <ojan> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ojan | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Tony Chang
2012-06-05 15:37:09 PDT
Created attachment 152287 [details]
Patch
Comment on attachment 152287 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=152287&action=review > Websites/bugs.webkit.org/PrettyPatch/PrettyPatch_test.rb:74 > + # FIXME: This should match, but there's a bug when running the tests where the image data > + # doesn't get properly written out the temp files, so we there there is no image and don't print Missing preposition? "...doesn't get properly written out [to] the temp files..." Extra 'we there'? "...so /we there/ there is no image..." > Websites/bugs.webkit.org/PrettyPatch/PrettyPatch_test.rb:75 > + # the warning that the image is missing it's checksum. s/it's/its/ Committed r122607: <http://trac.webkit.org/changeset/122607> |