Closed
Bug 376064
Opened 18 years ago
Closed 17 years ago
Back button doesn't work after document.open()
Categories
(Core :: DOM: Navigation, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jruderman, Assigned: bzbarsky)
References
(Blocks 1 open bug)
Details
(Keywords: testcase)
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
Biesinger
:
review+
jst
:
superreview+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
Steps to reproduce:
1. Load the testcase.
2. Click the Home button.
3. Click the Back button.
Result: Nothing happens when I click the Back button, even if I click it multiple times.
The testcase comes from bug 326621.
I'm testing with a Mac trunk debug build. I don't see any assertions or JavaScript errors related to this bug.
Reporter | ||
Updated•18 years ago
|
Flags: blocking1.9?
Updated•17 years ago
|
Flags: blocking1.9? → blocking1.9-
Reporter | ||
Comment 1•17 years ago
|
||
With a recent trunk build, something seems to happen when I click the Back button, but I'm still stuck.
Assignee | ||
Comment 2•17 years ago
|
||
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #294930 -
Flags: superreview?(jst)
Attachment #294930 -
Flags: review?(cbiesinger)
Updated•17 years ago
|
Attachment #294930 -
Flags: superreview?(jst) → superreview+
Updated•17 years ago
|
Attachment #294930 -
Flags: review?(cbiesinger) → review+
Assignee | ||
Comment 3•17 years ago
|
||
Comment on attachment 294930 [details] [diff] [review]
This does the trick
This just makes sure to not break session history if document.open() is called but then no data is written.
Attachment #294930 -
Flags: approval1.9?
Updated•17 years ago
|
Attachment #294930 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Comment 4•17 years ago
|
||
Fixed.
Assignee: bzbarsky → nobody
Status: ASSIGNED → NEW
Flags: in-testsuite?
Assignee | ||
Updated•17 years ago
|
Assignee: nobody → bzbarsky
Assignee | ||
Updated•17 years ago
|
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 5•17 years ago
|
||
The back button still gets stuck trying to go back past the document.open(), but I guess that's expected.
Assignee | ||
Comment 6•17 years ago
|
||
Right. You go back to that page, the script runs, and document.open() happens...
Component: History: Session → Document Navigation
QA Contact: history.session → docshell
You need to log in
before you can comment on or make changes to this bug.
Description
•