<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:ryuan.choi@navercorp.com" title="Ryuan Choi <ryuan.choi@navercorp.com>"> <span class="fn">Ryuan Choi</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - [EFL] Apply adjustScrollPositionWithinRange() to scroll offset for ScrollingCoordinatorCoordinatedGraphics."
href="https://bugs.webkit.org/show_bug.cgi?id=138083">bug 138083</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>ryuan.choi@navercorp.com
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [EFL] Apply adjustScrollPositionWithinRange() to scroll offset for ScrollingCoordinatorCoordinatedGraphics."
href="https://bugs.webkit.org/show_bug.cgi?id=138083#c10">Comment # 10</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [EFL] Apply adjustScrollPositionWithinRange() to scroll offset for ScrollingCoordinatorCoordinatedGraphics."
href="https://bugs.webkit.org/show_bug.cgi?id=138083">bug 138083</a>
from <span class="vcard"><a class="email" href="mailto:ryuan.choi@navercorp.com" title="Ryuan Choi <ryuan.choi@navercorp.com>"> <span class="fn">Ryuan Choi</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=138083#c4">comment #4</a>)
<span class="quote">> (In reply to <a href="show_bug.cgi?id=138083#c3">comment #3</a>)
> > Comment on <span class="bz_obsolete"><a href="attachment.cgi?id=240465&action=diff" name="attach_240465" title="Patch">attachment 240465</a> <a href="attachment.cgi?id=240465&action=edit" title="Patch">[details]</a></span>
> > Patch
> >
> > View in context:
> > <a href="https://bugs.webkit.org/attachment.cgi?id=240465&action=review">https://bugs.webkit.org/attachment.cgi?id=240465&action=review</a>
> >
> > >> Source/WebCore/ChangeLog:12
> > >> + No new tests because of no behavior change.
> > >
> > > I'm not sure this change won't change behavior. At least, this patch should pass WKViewScrollTo.cpp test.
> >
> > Can't you add new test in order to test this change ? It looks we need to
> > add a test which can handle when the offset value is not within the
> > scrollable range.
>
> Ok, let me check the WKViewScrollTo.cpp.
>
> However, this is a just bug fix for wheel scroll using mouse.
> Of course, you can reproduce issue easily using webkit.org.
> Just go to www.webkit.org and do the wheel scroll to the upper direction 10
> or more times, then do the wheel scroll to the down direction. you can
> notice that it will not scroll down for 10 times even though there is no
> webview changed.</span >
Why don't you make the test case like you mentioned?
In my two cents,
we can make the test case that scroll more than two times to invalid position and scroll few pixel (but valid).
I checked some use cases and reproduced this issue with very simple test cases that use windows.scrollTo and/or with fixed elements.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>