WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 3546
13122
No history added for a window.open followed by refresh
https://bugs.webkit.org/show_bug.cgi?id=13122
Summary
No history added for a window.open followed by refresh
Dex Deacon
Reported
2007-03-19 15:56:16 PDT
Repro steps: 1. Navigate to the attached test case. 2. Click the button to open a new window. 3. Navigate somewhere in the new window. Notice that you can't go back to the original page that opened the window. Firefox creates a history entry for the new page opened, Safari does not. I observed this in both Safari 2.0.4 and Safari running with WebKit
r20325
. From stepping through the code, it looks like the problem is in FrameLoader::updateHistoryForStandardLoad(). For the initial window.open(), the URL is empty, so history is not updated. Then the redirect occurs, and this function is called again, but this time it's rejected from history because it's a redirect.
Attachments
reduced test case
(254 bytes, text/html)
2007-03-19 15:57 PDT
,
Dex Deacon
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Dex Deacon
Comment 1
2007-03-19 15:57:21 PDT
Created
attachment 13707
[details]
reduced test case
Darin Fisher (:fishd, Google)
Comment 2
2007-04-18 15:40:16 PDT
The problem here is that no session history entry is created for either "about:blank" navigations or "client redirect" navigations. The <meta refresh> appears to be treated as a client redirect. I see this bug when I click a link in Gmail. IMO, this is a pretty major bug.
Brady Eidson
Comment 3
2007-04-18 15:45:22 PDT
I *think* this is a dupe of
http://bugs.webkit.org/show_bug.cgi?id=3546
which was fixed very recently
Brady Eidson
Comment 4
2007-04-18 15:48:56 PDT
Yep, its a dupe - this works on ToT. Closing.
David Kilzer (:ddkilzer)
Comment 5
2007-04-19 05:32:40 PDT
Reopening to mark as duplicate.
David Kilzer (:ddkilzer)
Comment 6
2007-04-19 05:32:54 PDT
*** This bug has been marked as a duplicate of
3546
***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug