[webkit-reviews] review granted: [Bug 57704] Remove AbstractStep._run_script and move script names to ports.py : [Attachment 87974] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sat Apr 2 04:48:22 PDT 2011
Eric Seidel <eric at webkit.org> has granted Patrick R. Gansterer
<paroga at paroga.com>'s request for review:
Bug 57704: Remove AbstractStep._run_script and move script names to ports.py
https://bugs.webkit.org/show_bug.cgi?id=57704
Attachment 87974: Patch
https://bugs.webkit.org/attachment.cgi?id=87974&action=review
------- Additional Comments from Eric Seidel <eric at webkit.org>
Sigh. Well, this is a good thing to do, but there are two troubles I should
point outin this patch. 1. ports.py should die. class methods aren't
long-term scaleable. We have a port.py over in layout_tests which we
eventually want to merge it with/replace it with.
Second, run_and_throw_if_fail also is deprecated and should die. :)
That said, this is a great change to get rid of a fixme!
I'm glad to see you workign on webkitpy!
More information about the webkit-reviews
mailing list