Bug 13378 - REGRESSION (r20809-r20814): History ignores first hyperlink step in certain windows
Summary: REGRESSION (r20809-r20814): History ignores first hyperlink step in certain w...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: 523.x (Safari 3)
Hardware: Macintosh OS X 10.4
: P2 Normal
Assignee: Nobody
URL: http://salokin.de/utf8
Keywords: HasReduction, InRadar, Regression
Depends on:
Blocks:
 
Reported: 2007-04-17 17:49 PDT by Nikolas Bertheau
Modified: 2007-12-10 06:34 PST (History)
1 user (show)

See Also:


Attachments
Reduction (783 bytes, text/html)
2007-04-19 16:29 PDT, Nikolas Bertheau
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nikolas Bertheau 2007-04-17 17:49:02 PDT
It is possible to load any page into a window with the following behavior: The first hyperlink step can't be undone (the back button stays blurred):
To reproduce, go to "http://salokin.de/utf8", press SPACE, enter any URL including "http://" and press ENTER. You should see a new window with the specified URL. Now follow any hyperlink. The back button stays blurred.
Comment 1 David Kilzer (:ddkilzer) 2007-04-17 22:05:19 PDT
Thanks Nikolas!  Confirmed with a local debug build of WebKit r20914 with Safari 2.0.4 (419.3) on Mac OS X 10.4.9 (8P135).

Why did you put "(800-814)" in the bug title?  We usually use numbers like that for the subversion revision that caused the regression.

Comment 2 David Kilzer (:ddkilzer) 2007-04-17 22:20:38 PDT
Other differences from shipping Safari (which may need separate bugs):

1. Background image doesn't load (or loads a different image).

2. A JavaScript error reported in the JavaScript console:

TypeError: Undefined value
http://salokin.de/utf8.html    Line: 51

Comment 3 Nikolas Bertheau 2007-04-18 01:06:32 PDT
(In reply to comment #1)
> Why did you put "(800-814)" in the bug title?  We usually use numbers like that
> for the subversion revision that caused the regression.

I remember putting there (809-814) meaning that until version r20809 everything was fine; problems started with version r20814.
Comment 4 Nikolas Bertheau 2007-04-18 02:12:37 PDT
(In reply to comment #2)
> Other differences from shipping Safari (which may need separate bugs):
> 
> 1. Background image doesn't load (or loads a different image).

No bug. This page uses a random choice of different backgrounds.
 
> 2. A JavaScript error reported in the JavaScript console:
> 
> TypeError: Undefined value
> http://salokin.de/utf8.html    Line: 51

No bug. Probably trying to focus a non-existing (blocked?) pop-up window.

Comment 5 David Kilzer (:ddkilzer) 2007-04-18 05:26:45 PDT
(In reply to comment #3)
> I remember putting there (809-814) meaning that until version r20809 everything
> was fine; problems started with version r20814.

Please add the whole revision number in the future.  Thanks!

Comment 6 Brady Eidson 2007-04-18 13:05:10 PDT
This is almost definitely caused by http://trac.webkit.org/projects/webkit/changeset/20813, which fixed http://bugs.webkit.org/show_bug.cgi?id=12005.  Ironically, that bug involved *too many* history items getting put in.

This bug, however, sounds a lot like http://bugs.webkit.org/show_bug.cgi?id=3546 (which was fixed in http://trac.webkit.org/projects/webkit/changeset/20878).

In that bug, gmail was found to be opening its new window in an extremely peculiar way that precluded the creation of that initial history item.  It seems likely that something similar is happening here.
Comment 7 Nikolas Bertheau 2007-04-19 16:29:53 PDT
Created attachment 14098 [details]
Reduction
Comment 8 Darin Adler 2007-04-23 08:38:55 PDT
<rdar://problem/5153022>