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

Chris Dumez
Reported 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.
Attachments
Patch (1.47 KB, patch)
2021-09-20 15:39 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2021-09-20 15:39:24 PDT
Created attachment 438741 [details] Patch
Geoffrey Garen
Comment 2 2021-09-20 16:47:44 PDT
Comment on attachment 438741 [details] Patch r=me
EWS
Comment 3 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].
Radar WebKit Bug Importer
Comment 4 2021-09-20 17:15:18 PDT
<rdar://problem/83331897>
Chris Dumez
Comment 5 2021-12-10 07:36:27 PST
Reverted r282794 for reason: Regressed PLT5 Committed r286850 (245083@trunk): <https://commits.webkit.org/245083@trunk>
Chris Dumez
Comment 6 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>
Note You need to log in before you can comment on or make changes to this bug.