Bug 146928 – Programmatic scrolling in RTL documents is no longer compatible with other browsers in WK2

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Simon Fraser (smfr)]

Created attachment 256756 [details]
Testcase

A programmatic document scroll in a RTL document behaves differently between WK2 and WK1, and between WK2 and Firefox. So WK2 broke compat with other browsers.

After:
                window.scrollTo(-50, 0);

window.scrollX is -870 in WK2, but -50 in WK1 and other browsers.

rdar://problem/21770741

[Simon Fraser (smfr)]

Created attachment 256757 [details]
WIP patch

[WebKit Commit Bot]

Attachment 256757 [details] did not pass style-queue:


ERROR: Source/WebCore/ChangeLog:8:  You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible.  [changelog/nonewtests] [5]
Total errors found: 1 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Build Bot]

Comment on attachment 256757 [details]
WIP patch

Attachment 256757 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/6567680114950144

Number of test failures exceeded the failure limit.

[Build Bot]

Created attachment 256758 [details]
Archive of layout-test-results from ews102 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-mavericks  Platform: Mac OS X 10.9.5

[Build Bot]

Comment on attachment 256757 [details]
WIP patch

Attachment 256757 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/4885281295564800

Number of test failures exceeded the failure limit.

[Build Bot]

Created attachment 256759 [details]
Archive of layout-test-results from ews106 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5

[Simon Fraser (smfr)]

Fixed in https://trac.webkit.org/changeset/194487