[Webkit-unassigned] [Bug 99842] Touch operation corrupts screen when specifying other than overflow:visible in css

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=99842

Ahmad Saleem <ahmad.saleem792 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ahmad.saleem792 at gmail.com

--- Comment #10 from Ahmad Saleem <ahmad.saleem792 at gmail.com> ---
(In reply to Brent Fulgham from comment #9)
> Comment on attachment 183879 [details]
> The modified patch to fix buildbot warning
> 
> I think this patch is good, but it would be better to include test cases so
> that we do not run into this again.  Could you look at what Terry Anderson
> did on the Chromium port (https://chromiumcodereview.appspot.com/14767007)
> and perhaps use the same test case?
> 
> Terry's change was at a lower level; perhaps we should do the same so that
> all ports would have the desired scrolling behavior, not just the Windows
> ports.

If we add "clamp" as third argument in this:

https://github.com/WebKit/WebKit/blob/71af797599da12fa56a502b6f23501f31e289dc2/Source/WebCore/rendering/RenderLayerScrollableArea.cpp#L1891

and merge test cases from the Chromium patch, will it work?

Tried to change testcase from Chromium patch into JSFiddle but it seems, it will work only in EWS:

https://jsfiddle.net/ca3pouzy/show

Or this is not needed anymore? Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220925/43d8e6b5/attachment-0001.htm>