Bug 230517

Summary: Avoid doing a second server pre-connect after a process-swap
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Chris Dumez <cdumez>
Status: REOPENED ---    
Severity: Normal CC: achristensen, ggaren, kkinnunen, nham, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Chris Dumez 2021-09-20 15:38:32 PDT 
Avoid doing a second server pre-connect after a process-swap. It is unnecessary as we've already asked the network process to do so before process-swapping.
Comment 1 Chris Dumez 2021-09-20 15:39:24 PDT 
Created attachment 438741 [details]
Patch
Comment 2 Geoffrey Garen 2021-09-20 16:47:44 PDT 
Comment on attachment 438741 [details]
Patch

r=me
Comment 3 EWS 2021-09-20 17:14:45 PDT 
Committed r282794 (241928@main): <https://commits.webkit.org/241928@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 438741 [details].
Comment 4 Radar WebKit Bug Importer 2021-09-20 17:15:18 PDT 
<rdar://problem/83331897>
Comment 5 Chris Dumez 2021-12-10 07:36:27 PST 
Reverted r282794 for reason:

Regressed PLT5

Committed r286850 (245083@trunk): <https://commits.webkit.org/245083@trunk>
Comment 6 Chris Dumez 2021-12-17 07:41:16 PST 
Reverted r286850 for reason:

Put r282794 back in. It turns out it was an overall progression after all

Committed r287185 (245352@trunk): <https://commits.webkit.org/245352@trunk>