1799169 - Fix early hint followed by redirect

Status: RESOLVED FIXED

(Core :: Networking: HTTP, defect, P2)

Description

[Manuel Bucher [:manuel]]

• https://searchfox.org/mozilla-central/source/netwerk/test/browser/browser_103_redirect_from_server.js could be in https://searchfox.org/mozilla-central/source/netwerk/test/browser/browser_103_redirect.js
• the redirect logic shouldnt happen in https://searchfox.org/mozilla-central/rev/eddb810ffd5499f0984123fe4bfea6064ebad3c8/netwerk/test/browser/early_hint_asset_html.sjs#9-10, but in https://searchfox.org/mozilla-central/rev/eddb810ffd5499f0984123fe4bfea6064ebad3c8/netwerk/test/browser/early_hint_redirect.sjs
• we still cancel the preload on same origin redirects? https://searchfox.org/mozilla-central/rev/eddb810ffd5499f0984123fe4bfea6064ebad3c8/netwerk/protocol/http/EarlyHintsService.cpp#88
• The test doesn't wait until the preload is complete -> intermittents.
• we need to make sure that we preload after cross origin redirects after https://phabricator.services.mozilla.com/D161181

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:kershaw, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 2

[Manuel Bucher [:manuel]]

Attachment: Bug 1799169 - Fix early hint tests in combination with redirects r=#necko

Depends on D161500

Comment 3

[Phabricator Automation]

Comment on attachment 9304484 [details]
Bug 1799169 - Fix early hint tests in combination with redirects r=#necko

Revision D162581 was moved to bug 1783686. Setting attachment 9304484 [details] to obsolete.

Comment 4

[Manuel Bucher [:manuel]]

Some of the notes were done in phase 2 preloader (Bug 1771867). The remaining notes will be fixed with Bug 1783686.