Bug 256626

Summary: FrameLoader::didFirstLayout() shouldn't restore scroll position and view state synchronously
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Layout and RenderingAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 256383    

Description Ryosuke Niwa 2023-05-10 21:09:16 PDT
It's problematic to restore scroll position and view state synchronously as that could involve nested layout / script execution.
Delay this work until Page::updateRendering.
Comment 1 Ryosuke Niwa 2023-05-10 21:13:29 PDT
Pull request: https://github.com/WebKit/WebKit/pull/13737
Comment 2 EWS 2023-05-10 23:12:11 PDT
Committed 263949@main (cc5ecdf646e3): <https://commits.webkit.org/263949@main>

Reviewed commits have been landed. Closing PR #13737 and removing active labels.
Comment 3 Radar WebKit Bug Importer 2023-05-10 23:13:20 PDT
<rdar://problem/109191090>