Bug 4072 – Pressing back in browser misses out a page.

• Component: New Bugs
• Status: RESOLVED FIXED

[David Storey]

Steps to reproduce:

Go to http://www.liverpoolfc.tv/  (It should redirect to a splash page if it is your first visit),
Click "Enter Liverpool FC TV", 
Click on a news item (Such as "WATCH TONIGHT'S GAME LIVE ON .TV"), 
Press back in the browser.

Expected output:

Should return to the homepage

Actual output:

Returns to the splash page, missing out the homepage.  

This works in Opera, Gecko and IE/Mac.  Note the the site uses a cookie so that the splash page only 
shows once a day, so it will not work if you have already visited the site today. 

trey2 on irc mentioned:  "part of the puzzle is that when you click on the "enter site" link, you go again 
to the same URL that you started with.  Safari sees that as the user going to the same place twice in a 
row, which doesn't create a second back/forward item.  Which still doesn't explain the misbehavior..I 
think the root of the problem is that the single b/f item has the resulting address after redirects have 
happened, and that doesn't get updated by the fact that the second visit to the same URL does not yield 
a redirect (because of the cookie)."

I will try to write a reduction in a while.

[Trey Matteson]

Created attachment 3024 [details]
proposed patch

[John Sullivan]

Comment on attachment 3024 [details]
proposed patch

Patch looks good. Please explain how you tested.

[Trey Matteson]

Primarily I tested the case described in the bug, as the fix is fairly specific to the odd case of the fetching 
the same URL resulting in arriving at 2 different final URLs (because of redirects that are dependend on 
cookies).  I also did some basic testing of refreshing a page and reloading a page, to look for major 
regressions.

[Darin Adler]

Comment on attachment 3024 [details]
proposed patch

Looks good to me, r=me.

[Darin Adler]

Comment on attachment 3024 [details]
proposed patch

Looks good, r=me.

[Justin Garcia]

Landing this...