Bug 152639 - Fix reload and programmatic scrolling in RTL documents
Summary: Fix reload and programmatic scrolling in RTL documents
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-01-01 14:02 PST by Simon Fraser (smfr)
Modified: 2016-01-01 20:50 PST (History)
5 users (show)

See Also:


Attachments
Patch (39.74 KB, patch)
2016-01-01 14:12 PST, Simon Fraser (smfr)
zalan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2016-01-01 14:02:25 PST
Fix reload and programmatic scrolling in RTL documents
Comment 1 Simon Fraser (smfr) 2016-01-01 14:12:32 PST
Created attachment 268075 [details]
Patch
Comment 2 Matt Baker 2016-01-01 16:38:54 PST
I've noticed other erratic scrolling behavior with elements using `overflow-y: auto`. Bisecting points to https://bugs.webkit.org/show_bug.cgi?id=152594 as the likely cause.

Reduction:

<style>
#content {
    width: 100px;
    height: 200px;
    overflow-y: auto;
}
</style>
<div id="content">
<p>Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec diam nisl, placerat non diam id, auctor facilisis tellus. In fermentum odio a consequat mattis.</p>
</div>

Vertically scrolling the paragraph causes accumulated errors in the scroll offset position. Hopefully this patch fixes it!
Comment 3 Simon Fraser (smfr) 2016-01-01 18:12:25 PST
Matt, this patch doesn't affect overflow scroll. Can you file a new bug on the issue you're seeing, with steps to reproduce?
Comment 4 Simon Fraser (smfr) 2016-01-01 18:52:19 PST
I can reproduce that bug and filed bug 152645.
Comment 5 Simon Fraser (smfr) 2016-01-01 20:50:22 PST
http://trac.webkit.org/changeset/194487