Bug 233168

Summary: Use IOHIDEvent timestamps for momentum velocity computation
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: ScrollingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: cmarcelo, ews-watchlist, fred.wang, jamesr, luiz, simon.fraser, thorton, tonikitoo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Simon Fraser (smfr) 2021-11-15 20:38:30 PST
Use IOHIDEvent timestamps for momentum velocity computation
Comment 1 Simon Fraser (smfr) 2021-11-15 20:40:17 PST
Created attachment 444338 [details]
Patch
Comment 2 Tim Horton 2021-11-15 21:51:55 PST
Comment on attachment 444338 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=444338&action=review

> Source/WebCore/page/scrolling/ThreadedScrollingTree.cpp:-340
> -    LOG_WITH_STREAM(ScrollAnimations, stream << "ThreadedScrollingTree::willStartRenderingUpdate - scrollingThreadIsActive " << scrollingThreadIsActive());

Where'd this go?
Comment 3 Simon Fraser (smfr) 2021-11-16 13:57:45 PST
(In reply to Tim Horton from comment #2)
> Comment on attachment 444338 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=444338&action=review
> 
> > Source/WebCore/page/scrolling/ThreadedScrollingTree.cpp:-340
> > -    LOG_WITH_STREAM(ScrollAnimations, stream << "ThreadedScrollingTree::willStartRenderingUpdate - scrollingThreadIsActive " << scrollingThreadIsActive());
> 
> Where'd this go?

It annoyed me so I removed it.
Comment 4 EWS 2021-11-16 14:05:08 PST
Committed r285883 (244308@main): <https://commits.webkit.org/244308@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 444338 [details].
Comment 5 Radar WebKit Bug Importer 2021-11-16 14:06:25 PST
<rdar://problem/85474517>