Bug 66446 - webkit-patch should have a command to mass-remove platform-dependent expected results
Summary: webkit-patch should have a command to mass-remove platform-dependent expected...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-17 19:05 PDT by Ryosuke Niwa
Modified: 2011-08-17 20:09 PDT (History)
4 users (show)

See Also:

Attachments
script (9.78 KB, application/octet-stream)
2011-08-17 19:06 PDT, Ryosuke Niwa 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2011-08-17 19:05:02 PDT 
Whenever I do a dump-as-markup conversions (e.g. bug 66416, bug 66420, bug 66440, bug 66443), I have to remove all platform-specific results and rebaseline test.  However this is quite tedious to do by hand because I'd have to find all expected.* in LayoutTests/platform for that particular test.  webkit-patch should have a command to do this.
Comment 1 Ryosuke Niwa 2011-08-17 19:06:26 PDT 
Created attachment 104295 [details]
script

Here's a python script I use locally but it only works on svn checkout as of now.
Comment 2 Adam Barth 2011-08-17 20:08:42 PDT 
If you look in http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/tool/commands/rebaseline.py you'll see a bunch of code that does similar things.  It should be easy to build something using those building blocks.
Comment 3 Adam Barth 2011-08-17 20:09:44 PDT 
Notice that http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/tool/commands/rebaseline.py#L128 can print a list of all these files, so you just need to call self._tool.filesystem.remove(...) on them.