<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:darin@apple.com" title="Darin Adler <darin@apple.com>"> <span class="fn">Darin Adler</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - Delegates scrolling by wheel event should be work within contents area."
href="https://bugs.webkit.org/show_bug.cgi?id=139757">bug 139757</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;">Attachment #252854 Flags</td>
<td>review?
</td>
<td>review-
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Delegates scrolling by wheel event should be work within contents area."
href="https://bugs.webkit.org/show_bug.cgi?id=139757#c21">Comment # 21</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Delegates scrolling by wheel event should be work within contents area."
href="https://bugs.webkit.org/show_bug.cgi?id=139757">bug 139757</a>
from <span class="vcard"><a class="email" href="mailto:darin@apple.com" title="Darin Adler <darin@apple.com>"> <span class="fn">Darin Adler</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=252854&action=diff" name="attach_252854" title="Patch">attachment 252854</a> <a href="attachment.cgi?id=252854&action=edit" title="Patch">[details]</a></span>
Patch
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=252854&action=review">https://bugs.webkit.org/attachment.cgi?id=252854&action=review</a>
review- because skipping this test on Mac only isn’t right. But we can’t see that problem with EWS because currently only Mac EWS runs tests, so skipping on Mac is enough to make EWS happy.
Someone needs to test this on iOS to see that it works as expected there. Could cause problems there because the scrolling UI involves rubber banding; this might interfere with that, maybe not.
<span class="quote">> LayoutTests/platform/mac/TestExpectations:287
> +# This port doesn't support delegate scrolling
> +fast/events/scroll-delegates-wheelevent.html [ WontFix ]</span >
This doesn’t seem right. Mac is not the only port that has this issue. It’s just that the EWS bot only runs the test on Mac. So skipping this only for Mac is incorrect. I bet this also doesn’t work on Windows, and perhaps some other platforms as well.
As far as I can tell, the “delegates scrolling” code path is used on iOS, and on platforms that use WebKit2 with COORDINATED_GRAPHICS. We shouldn’t be testing it on any other platforms unless we expect it to work.
So just skipping this on Mac is not right.</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>