Closed Bug 472632 Opened 16 years ago Closed 16 years ago

View source isn't reading from the cache

Categories

(Core Graveyard :: View Source, defect)

defect
Not set
major

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: neil, Assigned: neil)

References

Details

(Keywords: regression)

Attachments

(1 file)

As per bug 469302 and its duplicate, we need to QI the session history before we can add an entry to it.
Attached patch Proposed patchSplinter Review
Attachment #355928 - Flags: review?(iann_bugzilla)
Shouldn't this patch be identical to the one on 469302?
Comment on attachment 355928 [details] [diff] [review]
Proposed patch

r=me with just a comment in the bug as to why it's best to leave loadFromURL where it is.
Attachment #355928 - Flags: review?(iann_bugzilla) → review+
Pushed changeset 865f907bb16b to comm-central, with an extra comment change to indicate why I only set loadFromURL at the end of the try block.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Hmm, so if you add view-source: to the start of a URL (so you care about Back) after a post, and there's an error adding to session history, Firefox will show the source of the right page but Back will take you back to the form rather than the result of submitting it, while SeaMonkey will show the source of the form but Back will take you back to the result of submitting instead of to the form? Nice pair of tradeoffs.
Assignee: nobody → neil
Target Milestone: --- → seamonkey2.0a2
Product: SeaMonkey → Core Graveyard
Target Milestone: seamonkey2.0a2 → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: