New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 817387 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Deprecate blink provisional load concept

Project Member Reported by clamy@chromium.org, Feb 28 2018

Issue description

Post PlzNavigate the provisional load concept no longer makes sense. Ideally, we'd want to remove the concept entirely.

1) We should rename {Did|Will}CommitProvisionalLoad methods to something clearer: DidCommitNavigation and ReadyToCommitNavigation for example.
2) We should deprecate DidStartprovisionalLoad and DidFailProvisionalLoad as they are error-prone post PlzNavigate and follow up with teams still using them about their needs. We should see whether they can use existing navigation observers (e.g. in the browser process) or need to be specifically warned when a renderer requests a navigation from the browser process.
3) The end goal is to remove the provisional DoCumentLoader, which would simplify Blink's loading architecture.
 

Sign in to add a comment