frame location is updated too soon

[bridiver]

• Did you search for similar issues before submitting this one?
  yes

• Describe the issue you encountered:
  When navigating through the urlbar (and some other methods) the frame location is set immediately. This causes a problem for provisional load failures because they lose the original location to reset the urlbar. windowActions.loadUrl should set the provisional location and the actual location should be set on load-start. That allows certain types of did-fail-provisional-load (aborted for instance) failures to set the urlbar back to the provisionalLocation without reloading the page.

• Platform (Win7, 8, 10? macOS? Linux distro?):
  All

• Brave Version (revision SHA):
  396cdbc

[bridiver]

this information is sometimes available in the did-fail-provisional-load event, but that method uses GetURL and should use GetLastCommittedURL. GetURL returns the visible entry which should be correct after the load failure in DidFinishNavigation, but to be safe it should always be GetLastCommittedURL

[bridiver]

we could use that for this case, but what we're calling provisionalLocation in browser-laptop is not the provisional location which is really confusing because provisional load failures can only happen before load-start