Bug 4049 - scroll position not restored when going back/forward at ebay
Summary: scroll position not restored when going back/forward at ebay
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 412
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: John Sullivan
URL: http://ebay.com
Keywords:
Depends on:
Blocks:
 
Reported: 2005-07-18 12:04 PDT by John Sullivan
Modified: 2005-07-24 17:07 PDT (History)
1 user (show)

See Also:


Attachments
proposed patch - also fixes 4051 (6.24 KB, patch)
2005-07-18 16:55 PDT, Trey Matteson
no flags Details | Formatted Diff | Diff
proposed patch - also fixes 4061 (6.27 KB, patch)
2005-07-18 17:02 PDT, Trey Matteson
sullivan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John Sullivan 2005-07-18 12:04:47 PDT
This is also in Radar as 4005252

- go to ebay.com
- search for something
- scroll down at the search results page
- click on an item
- go back after the page for that item has loaded

result: you're back at the search results page, but the scroll position is not restored

This is makes searching for things at ebay very difficult.

Firefox does not have this problem.
Comment 1 Trey Matteson 2005-07-18 16:39:14 PDT
I stumbled onto a way to get the same behavior when I had some breakpoints set in gdb, at nytimes.com.  
I only rarely could make it happen at ebay, but it seems like a good chance that it is the same bug.  The 
flakey behavior is consistent with my analysis of it being a race condition.
Comment 2 Trey Matteson 2005-07-18 16:55:50 PDT
Created attachment 3010 [details]
proposed patch - also fixes 4051
Comment 3 Trey Matteson 2005-07-18 17:02:49 PDT
Created attachment 3013 [details]
proposed patch - also fixes 4061
Comment 4 John Sullivan 2005-07-19 08:35:04 PDT
Comment on attachment 3013 [details]
proposed patch - also fixes 4061

So this doesn't fix 4051 after all? Bummer. Still, nice fix.
Comment 5 John Sullivan 2005-07-19 10:28:26 PDT
I checked this in.
Comment 6 Maciej Stachowiak 2005-07-24 17:07:53 PDT
Comment on attachment 3010 [details]
proposed patch - also fixes 4051

Taking off review flag to make this stop showing up in the needs review query.