Open Bug 1852182 Opened 2 years ago Updated 2 years ago

Consider to remove doCreateAlternate path in nsDocShell to let the load to fail

Categories

(Core :: DOM: Navigation, enhancement)

enhancement

Tracking

()

People

(Reporter: daisuke, Unassigned)

References

Details

This is a follow-up of bug 1850902.

This is about the part of processing on nsDocShell::AttemptURIFixup.
As Marco suggested in phabricator, it now seems to fixup uri and retry loading if it fails to load, but it might be better, especially for the web developer, to have it fail to load.
If the uri fixup is not needed, we can remove the code below.
https://searchfox.org/mozilla-central/rev/146638366864f73ee8a697ea76480eb02c00eb3c/docshell/base/nsDocShell.cpp#6167-6207

We already disabled alternate fixup a few versions ago, so what the fixup is doing there is minor manipulation like fixing "://somehost" or "somehost..com"... that already happens in the address bar, so I think it'd be better to remove those magic transformations in the docshell.

You need to log in before you can comment on or make changes to this bug.