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 | ||
Ben Nham
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.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/106096729>