Summary: | LayoutTests/canvas/philip/tests/security.drawImage.image.html Failed and skipped | ||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chang Shu <cshu> | ||||||||||||||||||
Component: | DOM | Assignee: | Chang Shu <cshu> | ||||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||||
Severity: | Normal | CC: | cjerdonek, darin, dbates, eric, kenneth | ||||||||||||||||||
Priority: | P3 | ||||||||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||||||||
Hardware: | PC | ||||||||||||||||||||
OS: | All | ||||||||||||||||||||
Bug Depends on: | 42573, 49317 | ||||||||||||||||||||
Bug Blocks: | |||||||||||||||||||||
Attachments: |
|
Description
Chang Shu
2010-06-28 06:47:20 PDT
Created attachment 59900 [details]
fix patch
Created attachment 59902 [details]
fix patch
Created attachment 59911 [details]
fix patch
The script failed to patch the patch file but it seems to be a false-alarm to me. I don't understand why it tries to remove 'LayoutTests/canvas'. The log is here: fatal: not removing 'LayoutTests/canvas' recursively without -r Failed to git rm --force --ignore-unmatch LayoutTests/canvas. at /home/webkit/WebKit-qt-ews/WebKitTools/Scripts/svn-apply line 451. I suspect svn-apply is getting confused. https://trac.webkit.org/browser/trunk/WebKitTools/Scripts/VCSUtils.pm or https://trac.webkit.org/browser/trunk/WebKitTools/Scripts/svn-apply Am I the only one who has this problem? Is there a work-around so I can have my patch reviewed? thanks! Created attachment 61959 [details]
fix patch
The problem could be that in the patch, file tests.js has the wrong line-ending. I plan to commit a quick patch that does the following on this file: svn propset svn:eol-style native tests.js Does this sound ok? Created attachment 61968 [details]
fix patch
Created attachment 61971 [details]
fix patch
Created attachment 61976 [details]
fix patch 1
Created attachment 61983 [details]
fix patch 1
Darin, what do you think about this change? Comment on attachment 61983 [details]
fix patch 1
Seems OK to move these. But why are we skipping all these tests on all platforms?
(In reply to comment #14) > (From update of attachment 61983 [details]) > Seems OK to move these. But why are we skipping all these tests on all platforms? This is the 1st part of the patch. I split it because I thought it could solve the problem that my patch failed to be applied (see all the pink buttons above). Actually, the complete patch is attached already. I can re-enable it and if you think it's good, I will commit it manually. The 5th patch has the complete code change. Comment on attachment 61968 [details] fix patch manually committed attachment 61968 [details] webkit trunk r63752. |