Bug 253176

Summary: Unify preconnect implementations in NetworkProcess
Product: WebKit Reporter: Ben Nham <nham>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: nham, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   

Description Ben Nham 2023-03-01 11:20:19 PST
There are a lot of callsites in NetworkProcess that manually create a PreconnectTask with differing NetworkLoadParameters:

* NetworkProcess::preconnectTo - used to preconnect to main resources
* NetworkConnectionToWebProcess::preconnectTo - used to handle preconnects created by `<link rel="preconnect" ...>`
* SpeculativeLoadManager::preconnectForSubresource - used to preconnect to subresource domains speculatively

And then in 253120 I've added another callsite `EarlyHintsResourceLoader::startPreconnectTask`.

It seems like we should be able to unify these callsites into something a bit saner.
Comment 1 Radar WebKit Bug Importer 2023-03-01 11:20:50 PST
<rdar://problem/106096729>