Hi Sam, While I investigate this layout test failure in chromium, it looks to me that the windows baseline (LayoutTests/platform/win/fast/canvas/toDataURL-supportedTypes-expected.txt) is out of date and this test should not be skipped. The test for Windows used to fail but your patch (http://trac.webkit.org/changeset/35290) rewrote the test to not output the encoded image data, so it should pass now. I think we could remove this window specific baseline and fallback to the baseline in the test folder. You are the person who originally added the baseline, so I would like to confirm with you before changing it. I tested it locally and it works fine. Removing the Win baseline also fixes the issue in Chromium. Thanks, Victor
Created attachment 39998 [details] Proposed patch I think the win baseline for this test could be removed.
Comment on attachment 39998 [details] Proposed patch Nice find! We should wait a bit to give Sam a chance to speak up if there's something subtle here.
Everything looks good here. r=me as well.
Comment on attachment 39998 [details] Proposed patch Victor is not a committer, so adding cq+.
Comment on attachment 39998 [details] Proposed patch Rejecting patch 39998 from commit-queue. This patch will require manual commit. Patch https://bugs.webkit.org/attachment.cgi?id=39998 from bug 29619 failed to download and apply.
patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/platform/win/Skipped patching file LayoutTests/platform/win/fast/canvas/toDataURL-supportedTypes-expected.txt rm 'LayoutTests/platform/win/fast/canvas/toDataURL-supportedTypes-expected.txt' Could not open 'LayoutTests/platform/win/fast/canvas' to list files: 0 at /Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply line 358, <> line 99. That's a new one. It might be an error in svn-apply, not sure.
Reviewed patches have been landed. Closing bug