webkitpy.port supports layout tests and API tests, among other things. The factory code is imported all over the place, so this turns out to be a blocker for converting the rest of webkitpy.common.
Created attachment 382077 [details] Patch
Comment on attachment 382077 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382077&action=review > Tools/Scripts/webkitpy/common/version.py:122 > + result ^= hash(self[i]) This may not be accurate and easy to create a collision. For example "1.2.3.4.5" will have the same has as "5.4.3.2.1" if I understand the hash algorithm correctly here. How about using `hash(self.__str__())` or `hash(str(self))` instead?
Created attachment 382339 [details] Patch
Comment on attachment 382339 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=382339&action=review > Tools/ChangeLog:26 > + * Scripts/webkitpy/port/config_standalone.py: Ditto Please update the change description for this line. It was a bug before this change.
Created attachment 382353 [details] Patch for landing
Comment on attachment 382353 [details] Patch for landing Clearing flags on attachment: 382353 Committed r251808: <https://trac.webkit.org/changeset/251808>
All reviewed patches have been landed. Closing bug.
<rdar://problem/56759792>