Summary: | [Qt] LaytoutTests/fast/canvas/fill-stroke-clip-reset-path.html failed and skipped | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chang Shu <cshu> | ||||||
Component: | New Bugs | Assignee: | Chang Shu <cshu> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, hausmann, kenneth, laszlo.gombos | ||||||
Priority: | P3 | Keywords: | Qt | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | Linux | ||||||||
Attachments: |
|
Description
Chang Shu
2010-03-30 18:37:22 PDT
Created attachment 52116 [details]
fix patch
Created attachment 52119 [details]
fix patch 2
I'd like to get more feedback on what should be the policy for tests are are testing unsupported features. I can see two options here: 1./ Create a section in the skipped list for unsupported features that we do not intend to fix as a conscious decision has been made to not to support (or obsolete) a particular feature 2./ Check in the expected failure as Chang suggested, so that we can monitor for regression going froward. Comment on attachment 52119 [details]
fix patch 2
r+. Let's land the expected results even if the feature is not supported to catch regressions.
Comment on attachment 52119 [details] fix patch 2 Rejecting patch 52119 from commit-queue. Failed to run "['/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', '--reviewer', 'Laszlo Gombos', '--force']" exit_code: 1 Last 500 characters of output: ng file LayoutTests/platform/qt/Skipped Hunk #1 FAILED at 2567. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/qt/Skipped.rej patching file LayoutTests/platform/qt/fast/canvas/fill-stroke-clip-reset-path-expected.checksum patching file LayoutTests/platform/qt/fast/canvas/fill-stroke-clip-reset-path-expected.txt error: pathspec 'LayoutTests/platform/qt/fast/canvas/fill-stroke-clip-reset-path-expected.png' did not match any file(s) known to git. Did you forget to 'git add'? Full output: http://webkit-commit-queue.appspot.com/results/1624148 Merged the code and committed patch manually. Committed revision 56911. |