Regression(r246188) WebProcess is launched too eagerly when [WKWebView _restoreSessionState] is called

[Chris Dumez]

Since r246188, the WebProcess is launched eagerly when [WKWebView _restoreSessionState] is called. This is bad for performance because we are unable to leverage the process cache at this point (since we don't know which domain will be loaded).

[Chris Dumez]

Created attachment 391082 [details]
Patch

[Geoffrey Garen]

Comment on attachment 391082 [details]
Patch

Seems like the api test failure is real.

[Chris Dumez]

Comment on attachment 391082 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=391082&action=review

> Source/WebKit/UIProcess/WebPageProxy.cpp:-3404
> -        m_backForwardList->restoreFromState(WTFMove(sessionState.backForwardListState));

lol, I dropped this line by mistake :o)

[Chris Dumez]

Created attachment 391189 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 391189 [details]
Patch

Clearing flags on attachment: 391189

Committed r257045: <https://trac.webkit.org/changeset/257045>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/59619323>

[WebKit Commit Bot]

Re-opened since this is blocked by bug 208024

[Chris Dumez]

Created attachment 394926 [details]
Patch

[EWS]

Committed r259209: <https://trac.webkit.org/changeset/259209>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 394926 [details].