Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: ac5860080b7db9ca627ac8b2f17d67214c40c865
https://github.com/WebKit/WebKit/commit/ac5860080b7db9ca627ac8b2f17d67214c40c865
Author: Charlie Wolfe <[email protected]>
Date: 2025-05-12 (Mon, 12 May 2025)
Changed paths:
A LayoutTests/http/tests/history/redirect-302-twice-expected.txt
A LayoutTests/http/tests/history/redirect-302-twice.html
M LayoutTests/http/tests/history/resources/redirect-helper.pl
M LayoutTests/http/tests/history/resources/redirect-target.html
M Source/WebCore/loader/FrameLoader.cpp
Log Message:
-----------
xkcd.com: Back navigation doesn't work after using the "random" button
https://bugs.webkit.org/show_bug.cgi?id=292739
rdar://90175606
Reviewed by Alex Christensen.
This condition was added 22 years ago in 2796@main and I’m not sure why. It’s
causing an issue where,
if subsequent navigations to the same URL result in a 302 redirect to a new
URL, the new URL is not added
to the back/forward list. This change aligns Safari’s behavior with Chrome and
Firefox.
* LayoutTests/http/tests/history/redirect-302-twice-expected.txt: Added.
* LayoutTests/http/tests/history/redirect-302-twice.html: Added.
* LayoutTests/http/tests/history/resources/redirect-helper.pl:
* LayoutTests/http/tests/history/resources/redirect-target.html:
* Source/WebCore/loader/FrameLoader.cpp:
(WebCore::FrameLoader::shouldTreatURLAsSameAsCurrent const):
Canonical link: https://commits.webkit.org/294798@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
