Bug 4049

Summary:

scroll position not restored when going back/forward at ebay

Product:

WebKit

Reporter:

John Sullivan <sullivan>

Component:

WebKit Misc.

Assignee:

John Sullivan <sullivan>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

trey

Priority:

Version:

412

Hardware:

Mac

OS:

OS X 10.4

URL:

http://ebay.com

Attachments:

Description	Flags
proposed patch - also fixes 4051	none
proposed patch - also fixes 4061	sullivan: review+

John Sullivan

Reported 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.

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
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Trey Matteson

Comment 1 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.

Trey Matteson

Comment 2 2005-07-18 16:55:50 PDT

Created attachment 3010 [details] proposed patch - also fixes 4051

Trey Matteson

Comment 3 2005-07-18 17:02:49 PDT

Created attachment 3013 [details] proposed patch - also fixes 4061

John Sullivan

Comment 4 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.

John Sullivan

Comment 5 2005-07-19 10:28:26 PDT

I checked this in.

Maciej Stachowiak

Comment 6 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.

Note You need to log in before you can comment on or make changes to this bug.