Make port/config.py mockable for easier unit testing
Created attachment 99154 [details] Patch
Comment on attachment 99154 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=99154&action=review > Tools/Scripts/webkitpy/common/system/filesystem.py:62 > + import sys Why not import sys at the top of the file?
Comment on attachment 99154 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=99154&action=review >> Tools/Scripts/webkitpy/common/system/filesystem.py:62 >> + import sys > > Why not import sys at the top of the file? Sure, happy to move it.
Sigh. This landed as http://trac.webkit.org/changeset/90050. Silly git.
Looks fine to me also ...
(In reply to comment #5) > Looks fine to me also ... Thanks for looking!
Fixed in http://trac.webkit.org/changeset/90051.