Bug 256626 - FrameLoader::didFirstLayout() shouldn't restore scroll position and view state synchronously
Summary: FrameLoader::didFirstLayout() shouldn't restore scroll position and view stat...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords: InRadar
Depends on:
Blocks: 256383
  Show dependency tree / graph
 
Reported: 2023-05-10 21:09 PDT by Ryosuke Niwa
Modified: 2023-05-10 23:13 PDT (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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>