Bug 152639

Summary: Fix reload and programmatic scrolling in RTL documents
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, mattbaker, simon.fraser, thorton, zalan
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch zalan: review+

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