Bug 195733

Summary: [Async overflow scrolling] Update positioned node layers when overflows are scrolled
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ews-watchlist, fred.wang, koivisto, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 195780    
Bug Blocks:    
Attachments:
Description Flags
Code ready, needs tests.
none
Code ready, needs tests
none
Code ready, needs tests
none
Archive of layout-test-results from ews125 for ios-simulator-wk2
none
Archive of layout-test-results from ews200 for win-future
none
Patch none

Description Simon Fraser (smfr) 2019-03-13 20:36:47 PDT
[Coordinated Scrolling] Update positioned node layers when overflows are scrolled
Comment 1 Simon Fraser (smfr) 2019-03-13 20:44:55 PDT Comment hidden (obsolete)
Comment 2 Simon Fraser (smfr) 2019-03-13 20:44:57 PDT
<rdar://problem/11642295>
Comment 3 Simon Fraser (smfr) 2019-03-13 20:51:49 PDT Comment hidden (obsolete)
Comment 4 Simon Fraser (smfr) 2019-03-13 21:04:29 PDT Comment hidden (obsolete)
Comment 5 EWS Watchlist 2019-03-13 23:00:04 PDT Comment hidden (obsolete)
Comment 6 EWS Watchlist 2019-03-13 23:00:06 PDT Comment hidden (obsolete)
Comment 7 EWS Watchlist 2019-03-14 00:46:55 PDT Comment hidden (obsolete)
Comment 8 EWS Watchlist 2019-03-14 00:47:06 PDT Comment hidden (obsolete)
Comment 9 Simon Fraser (smfr) 2019-03-14 17:54:21 PDT
Created attachment 364737 [details]
Patch
Comment 10 Antti Koivisto 2019-03-15 08:50:37 PDT
Comment on attachment 364737 [details]
Patch

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

r=me

> Source/WebCore/page/scrolling/cocoa/ScrollingTreePositionedNode.mm:80
> +    // Note that we ignore cumulativeDelta because it will contain the delta for ancestor scrollers,
> +    // but not non-ancestor ones, so it's simpler to just recompute from the scrollers we know about here.

Would be nice to have a single unified way to do this everywhere.
Comment 11 WebKit Commit Bot 2019-03-15 09:26:15 PDT
Comment on attachment 364737 [details]
Patch

Clearing flags on attachment: 364737

Committed r242997: <https://trac.webkit.org/changeset/242997>
Comment 12 WebKit Commit Bot 2019-03-15 09:26:17 PDT
All reviewed patches have been landed.  Closing bug.