Bug 264229

Summary: markAllBackingStoreVolatileFromTimer can stop the volatility timer too soon
Product: WebKit Reporter: Matt Woodrow <mattwoodrow>
Component: Layout and RenderingAssignee: Matt Woodrow <mattwoodrow>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Description Matt Woodrow 2023-11-05 12:47:04 PST 
<rdar://116711724>

RemoteLayerWithRemoteRenderingBackingStoreCollection::markAllBackingStoreVolatileFromTimer stops the volatility timer if the operation is successful, since there should be no further backing stores to mark volatile.

This is an async operation though, and it's possible for markBackingStoreVolatileAfterReachabilityChange to have been called in the interim and there are now new backing stores that aren't yet volatile.
Comment 1 Matt Woodrow 2023-11-05 12:49:39 PST 
Pull request: https://github.com/WebKit/WebKit/pull/20014
Comment 2 EWS 2023-11-09 15:50:36 PST 
Committed 270486@main (a6666b946bcc): <https://commits.webkit.org/270486@main>

Reviewed commits have been landed. Closing PR #20014 and removing active labels.