Bug 164491 - LayoutTest fast/visual-viewport/rtl-zoomed-rects.html failing
Summary: LayoutTest fast/visual-viewport/rtl-zoomed-rects.html failing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-11-07 15:02 PST by Ryan Haddad
Modified: 2016-11-09 11:24 PST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2016-11-07 15:02:07 PST
LayoutTest fast/visual-viewport/rtl-zoomed-rects.html failing

https://build.webkit.org/results/Apple%20Yosemite%20Release%20WK1%20(Tests)/r208422%20(19547)/results.html

http://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fvisual-viewport%2Frtl-zoomed-rects.html

--- /Volumes/Data/slave/yosemite-release-tests-wk1/build/layout-test-results/fast/visual-viewport/rtl-zoomed-rects-expected.txt
+++ /Volumes/Data/slave/yosemite-release-tests-wk1/build/layout-test-results/fast/visual-viewport/rtl-zoomed-rects-actual.txt
@@ -3,7 +3,7 @@
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
-JSON.stringify(internals.layoutViewportRect()) is {"top":0,"right":392.5,"bottom":585,"left":-392.5,"width":785,"height":585}
+JSON.stringify(internals.layoutViewportRect()) is {"top":0,"right":785,"bottom":585,"left":0,"width":785,"height":585}
 JSON.stringify(internals.visualViewportRect()) is {"top":0,"right":392.5,"bottom":292.5,"left":0,"width":392.5,"height":292.5}
 
 Scrolled to -475, 525
Comment 1 Ryan Haddad 2016-11-07 15:02:20 PST
Added with https://trac.webkit.org/changeset/208409
Comment 2 Ryan Haddad 2016-11-07 15:04:51 PST
This test is failing on Yosemite and El Capitan WK1

https://build.webkit.org/results/Apple%20El%20Capitan%20Release%20WK1%20(Tests)/r208422%20(11293)/results.html
Comment 3 Alexey Proskuryakov 2016-11-09 10:29:48 PST
This is making EWS substantially slower, which is a especially serous problem now that everyone is trying to land accumulated patches.

If there is no comment from Simon, please roll out ASAP.
Comment 4 Simon Fraser (smfr) 2016-11-09 10:34:45 PST
I'll land WK1 results. WK1 is known to have issues with RTL scrolling.
Comment 5 Simon Fraser (smfr) 2016-11-09 11:21:04 PST
The test passes on Sierra. Going to make as failing on earlier.
Comment 6 Simon Fraser (smfr) 2016-11-09 11:24:09 PST
https://trac.webkit.org/changeset/208453