Bug 4061 - When going back/forward to some pages, they redraw at top before restoring scroll position
Summary: When going back/forward to some pages, they redraw at top before restoring sc...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 412
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: John Sullivan
URL: http://www.nytimes.com
Keywords:
Depends on:
Blocks:
 
Reported: 2005-07-18 14:51 PDT by Trey Matteson
Modified: 2005-07-19 16:48 PDT (History)
0 users

See Also:


Attachments
also fixes 4049 (6.24 KB, patch)
2005-07-18 16:59 PDT, Trey Matteson
no flags Details | Formatted Diff | Diff
proposed patch, also fixes 4049 (6.27 KB, patch)
2005-07-18 17:01 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 Trey Matteson 2005-07-18 14:51:02 PDT
Go to nytimes.com.  Scroll down a page or two.  Go to about:blank (or anywhere else).   Go back.

The nytimes page redraws first at the top of the screen, then jumps down the the scroll position.  Ideally it 
would just draw once at the final location without the jump.

This was also Radar 3024377, which was closed as non-repro.  I can repro it at nytimes.com now.  One 
change that might have caused the closing is that the bug does not happen for pages that use the b/f 
cache, so that might have fixed the original case.  nytimes has frames, so it does not use the b/f cache.
Comment 1 Trey Matteson 2005-07-18 16:59:30 PDT
Created attachment 3011 [details]
also fixes 4049
Comment 2 Trey Matteson 2005-07-18 17:01:37 PDT
Created attachment 3012 [details]
proposed patch, also fixes 4049
Comment 3 John Sullivan 2005-07-19 10:31:52 PDT
Checked this in.
Comment 4 John Sullivan 2005-07-19 16:48:57 PDT
Comment on attachment 3012 [details]
proposed patch, also fixes 4049

I already marked this review+ for bug 4049; doing so here also so it doesn't
show up in queries.