RESOLVED FIXED 132737
[CSS Blending] Adjust blending pixel test expectations 
https://bugs.webkit.org/show_bug.cgi?id=132737
Summary [CSS Blending] Adjust blending pixel test expectations
Ion Rosca
Reported 2014-05-09 05:09:28 PDT
[CSS Blending] Adjust blending test expectations after removing the -webkit- prefix
Attachments
Patch (3.08 MB, patch)
2014-05-09 05:18 PDT, Ion Rosca 		no flags
DetailsFormatted DiffDiff
Patch (3.08 MB, patch)
2014-05-09 05:53 PDT, Ion Rosca 		no flags
DetailsFormatted DiffDiff
Patch (3.08 MB, patch)
2014-05-09 06:05 PDT, Ion Rosca 		no flags
DetailsFormatted DiffDiff
Patch (3.08 MB, patch)
2014-05-12 01:36 PDT, Ion Rosca 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Ion Rosca
Comment 1 2014-05-09 05:18:24 PDT
Created attachment 231145 [details] Patch
Ion Rosca
Comment 2 2014-05-09 05:53:41 PDT
Created attachment 231147 [details] Patch
WebKit Commit Bot
Comment 3 2014-05-09 05:56:09 PDT
Attachment 231147 [details] did not pass style-queue: ERROR: LayoutTests/ChangeLog:11: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:12: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:14: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:15: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:24: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:25: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:26: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:44: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:45: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:46: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] ERROR: LayoutTests/ChangeLog:47: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5] Total errors found: 11 in 8 files If any of these errors are false positives, please file a bug against check-webkit-style.
Ion Rosca
Comment 4 2014-05-09 06:05:16 PDT
Created attachment 231149 [details] Patch
Mihnea Ovidenie
Comment 5 2014-05-12 00:18:24 PDT
Comment on attachment 231149 [details] Patch r=me
WebKit Commit Bot
Comment 6 2014-05-12 00:22:08 PDT
Comment on attachment 231149 [details] Patch Rejecting attachment 231149 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 231149, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: ode-image-color-dynamic-expected.html (working copy) -------------------------- No file to patch. Skipping patch. 1 out of 1 hunk ignored patching file LayoutTests/platform/mac-wk1/TestExpectations patching file LayoutTests/platform/mac/TestExpectations patching file LayoutTests/platform/mac/css3/blending/blend-mode-overflow-expected.txt Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Mihnea Ovidenie']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.appspot.com/results/5306088312274944
Ion Rosca
Comment 7 2014-05-12 01:36:38 PDT
Created attachment 231278 [details] Patch
WebKit Commit Bot
Comment 8 2014-05-12 02:22:48 PDT
Comment on attachment 231278 [details] Patch Clearing flags on attachment: 231278 Committed r168616: <http://trac.webkit.org/changeset/168616>
WebKit Commit Bot
Comment 9 2014-05-12 02:22:51 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.