Bug 214627

Summary: Scrolling tesla.com on iPad is not smooth
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: ScrollingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: amit23.lfc, cdumez, cmarcelo, ews-watchlist, fred.wang, jamesr, koivisto, krollin, luiz, simon.fraser, thorton, tonikitoo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=173887
Attachments:
Description Flags
Patch thorton: review+

Description Simon Fraser (smfr) 2020-07-21 22:04:09 PDT
Scrolling tesla.com on iPad is not smooth
Comment 1 Simon Fraser (smfr) 2020-07-21 22:09:02 PDT
Created attachment 404900 [details]
Patch
Comment 2 Simon Fraser (smfr) 2020-07-21 22:09:05 PDT
<rdar://problem/65568979>
Comment 3 Tim Horton 2020-07-22 09:51:37 PDT
Comment on attachment 404900 [details]
Patch

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

Sbould make sure that a page that goes into the page cache with an active scroll comes out of it without one, probably?

> Source/WebKit/Shared/RemoteLayerTree/RemoteScrollingUIState.h:50
> +    RemoteScrollingUIState() = default;

Isn't this line needless?
Comment 4 Tim Horton 2020-07-22 09:52:03 PDT
(or the process-swappy-cache, which ... might be the same?)
Comment 5 Chris Dumez 2020-07-22 10:04:02 PDT
(In reply to Tim Horton from comment #4)
> (or the process-swappy-cache, which ... might be the same?)

It is called back/forward cache now and is the same whether you proses-swap or not :)
Comment 6 Chris Dumez 2020-07-22 10:06:51 PDT
Cc'ing Keith because he cares :)
Comment 7 Simon Fraser (smfr) 2020-07-22 10:48:54 PDT
https://trac.webkit.org/changeset/264704/webkit
Comment 8 Simon Fraser (smfr) 2020-08-13 21:11:16 PDT
*** Bug 215254 has been marked as a duplicate of this bug. ***