Closed
Bug 130931
Opened 24 years ago
Closed 24 years ago
[FIX]view-source follows refresh header, and shows the new page
Categories
(Core :: Networking, defect, P1)
Tracking
()
VERIFIED
FIXED
mozilla1.0
People
(Reporter: mvl, Assigned: bzbarsky)
References
()
Details
Attachments
(1 file)
|
926 bytes,
patch
|
doronr
:
review+
darin.moz
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.9+) Gecko/20020313
BuildID: 2002031321
When visiting a page with a refresh: header, view-source also goes to the url in
the header.
Reproducible: Always
Steps to Reproduce:
1. visit http://www.shop-engine.nl/moztest/redir.php
2. within 5 seconds, view the source
3. wait a couple of seconds
Actual Results: Window with source also show bugzilla.mozilla.org
Expected Results: Window with source displays the source of the page
The mentioned url contains this php code:
<?
header("Refresh: 5; url=http://bugzilla.mozilla.org");
?>
|
Assignee | |
Comment 1•24 years ago
|
||
|
|
Assignee | |
Comment 2•24 years ago
|
||
taking bug
Assignee: new-network-bugs → bzbarsky
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: review
Priority: -- → P1
Summary: view-source follows refresh header, and shows the new page → [FIX]view-source follows refresh header, and shows the new page
Target Milestone: --- → mozilla1.0
|
||
Comment 3•24 years ago
|
||
Comment on attachment 74154 [details] [diff] [review]
Patch to fix
r=doron
Attachment #74154 -
Flags: review+
|
|
||
Comment 4•24 years ago
|
||
Comment on attachment 74154 [details] [diff] [review]
Patch to fix
sr=darin
Attachment #74154 -
Flags: superreview+
|
||
Comment 5•24 years ago
|
||
Comment on attachment 74154 [details] [diff] [review]
Patch to fix
a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #74154 -
Flags: approval+
|
|
Assignee | |
Comment 6•24 years ago
|
||
Checked in. Thanks for the bug report and the excellent testcase, Michiel!
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
VERIFIED: Mozilla 1.1
So you want it to say:
<html>
<body>
refresh in 5 sec
</body>
</html>
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•