Add Python 3 support to layout test logging.
Created attachment 383650 [details] Patch
Comment on attachment 383650 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=383650&action=review LGTM overall w/ a question. Unofficial r+. > Tools/Scripts/webkitpy/layout_tests/views/printing_unittest.py:169 > + printer._meter.number_of_columns = lambda: sys.maxsize Does this exist in both py2 and py3?
(In reply to Dean Johnson from comment #2) > Comment on attachment 383650 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=383650&action=review > > LGTM overall w/ a question. Unofficial r+. > > > Tools/Scripts/webkitpy/layout_tests/views/printing_unittest.py:169 > > + printer._meter.number_of_columns = lambda: sys.maxsize > > Does this exist in both py2 and py3? Yes. maxint doesn't exist in Python 3, maxsize is 9223372036854775807 in both Python 2 and Python 3, maxint is also 9223372036854775807 in Python 2.
Comment on attachment 383650 [details] Patch Does it passing webkitpy ews indicates that these updated tests were run through EWS (for Python 2)?
(In reply to Aakash Jain from comment #4) > Comment on attachment 383650 [details] > Patch > > Does it passing webkitpy ews indicates that these updated tests were run > through EWS (for Python 2)? Yes! And actually, the new tests are pretty similar to the old ones. The big difference is that we were essentially introspecting buflist on the StringIO object so avoid concatenating unicode and non-unicode strings. In most cases, we could safely join the StringIO buffers (and we can always join them in Python 3) so with a few notable unicode exceptions, that's what the new unit tests are doing. Some added comfort here, we passed api and layout tests which are also leveraging Python 2 webkitpy code.
The commit-queue encountered the following flaky tests while processing attachment 383650 [details]: imported/w3c/web-platform-tests/svg/import/struct-dom-06-b-manual.svg bug 204312 (author: sabouhallawa@apple.com) The commit-queue is continuing to process your patch.
Comment on attachment 383650 [details] Patch Clearing flags on attachment: 383650 Committed r252560: <https://trac.webkit.org/changeset/252560>
All reviewed patches have been landed. Closing bug.
<rdar://problem/57293904>