RESOLVED FIXED 179345
[WinCairo] Treat 'future' version correctly
https://bugs.webkit.org/show_bug.cgi?id=179345
Summary [WinCairo] Treat 'future' version correctly
Basuke Suzuki
Reported 2017-11-06 14:22:06 PST
It just causes error currently if os_version detection falls into 'future'.
Attachments
patch (1.66 KB, patch)
2017-11-07 17:49 PST, Basuke Suzuki
buildbot: commit-queue-
Archive of layout-test-results from ews124 for ios-simulator-wk2 (2.23 MB, application/zip)
2017-11-07 19:57 PST, Build Bot
no flags
patch (1.35 KB, patch)
2017-11-09 12:12 PST, Basuke Suzuki
no flags
Basuke Suzuki
Comment 1 2017-11-07 17:49:44 PST
Basuke Suzuki
Comment 2 2017-11-07 17:51:12 PST
WinPort has many adhoc implementations. To make the original purpose of those method clear, WinCairo is starting over with ApplePort which has basic OS version handling. Upcoming patches will make the method of WinPort to be usable from WinCairoPort by making those separate library or at least @static.
Build Bot
Comment 3 2017-11-07 19:57:58 PST
Comment on attachment 326280 [details] patch Attachment 326280 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/5141993 New failing tests: http/tests/workers/service/service-worker-clear.html
Build Bot
Comment 4 2017-11-07 19:57:59 PST
Created attachment 326293 [details] Archive of layout-test-results from ews124 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews124 Port: ios-simulator-wk2 Platform: Mac OS X 10.12.6
Basuke Suzuki
Comment 5 2017-11-09 12:12:18 PST
Basuke Suzuki
Comment 6 2017-11-09 12:13:14 PST
Stop reconstructing the port class from the scratch. Just fix the issue.
WebKit Commit Bot
Comment 7 2017-11-10 16:18:34 PST
Comment on attachment 326476 [details] patch Clearing flags on attachment: 326476 Committed r224713: <https://trac.webkit.org/changeset/224713>
WebKit Commit Bot
Comment 8 2017-11-10 16:18:35 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 9 2017-11-15 09:42:33 PST
Note You need to log in before you can comment on or make changes to this bug.