153764 2016-02-01 13:43:31 -0800 [iOS Simulator] fast/viewport/viewport-warnings-7.html failing 2016-02-02 22:28:03 -0800 1 1 1 Unclassified WebKit Tools / Tests WebKit Nightly Build iPhone / iPad iOS 9.0 RESOLVED FIXED P2 Normal --- 152130 1 ryanhaddad webkit-unassigned ap darin lforschler oldest_to_newest 1161010 0 ryanhaddad 2016-02-01 13:43:31 -0800 [iOS Simulator] fast/viewport/viewport-warnings-7.html failing <https://build.webkit.org/results/Apple%20iOS%209%20Simulator%20Release%20WK2%20(Tests)/r195975%20(2771)/results.html> <http://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fviewport%2Fviewport-warnings-7.html> --- /Volumes/Data/slave/ios-simulator-9-release-tests-wk2/build/layout-test-results/fast/viewport/viewport-warnings-7-expected.txt +++ /Volumes/Data/slave/ios-simulator-9-release-tests-wk2/build/layout-test-results/fast/viewport/viewport-warnings-7-actual.txt @@ -1,3 +1,3 @@ -CONSOLE MESSAGE: line 5: Viewport argument value "123i̇" for key "width" was truncated to its numeric prefix. +CONSOLE MESSAGE: line 5: Viewport argument value "123İ" for key "width" was truncated to its numeric prefix. ALERT: viewport size 123x135.3 scale 2.60163 with limits [2.60163, 5] and userScalable true 1161011 1 ryanhaddad 2016-02-01 13:45:10 -0800 Failing since <http://trac.webkit.org/changeset/195951> 1161535 2 ap 2016-02-02 21:38:13 -0800 This is a progression, please feel free to just update iOS results. That said, it seems wrong that we log a transformed variant of the value instead of the original. 1161542 3 darin 2016-02-02 21:46:31 -0800 I believe the old code was logging a transformed variant of the value. The new code logs the original. That’s what the progression is. 1161543 4 darin 2016-02-02 21:46:56 -0800 Ryan, will you update the iOS expected result or would you like me to do it? 1161547 5 darin 2016-02-02 21:56:55 -0800 Yes, confirmed. This new result logs the original value. So the progression is not because we longer lowercase non-ASCII characters; it’s because the new code doesn’t lowercase anything at all, and logs the original value. 1161557 6 darin 2016-02-02 22:28:03 -0800 http://trac.webkit.org/changeset/196047