|Summary:||REGRESSION (r19631): Going back one page in history has a noticeable delay|
|Product:||WebKit||Reporter:||David Kilzer (:ddkilzer) <ddkilzer>|
|Component:||Page Loading||Assignee:||Nobody <webkit-unassigned>|
|OS:||OS X 10.4|
Description David Kilzer (:ddkilzer) 2007-02-16 07:10:40 PST
Summary: When using Cmd-Left Arrow or using the Safari browser back button to go back one page in history, there is a noticeable (and annoying) delay in recent nightly/ToT builds. Steps to reproduce: 1. Open Safari/WebKit. 2. Open URL with lots of links: http://lists.macosforge.org/pipermail/webkit-unassigned/2007-February/date.html 3. Click on a link (any link). 4. Hit the browser back button (or Cmd-Left Arrow). Expected results: The previous page should be displayed in a reasonable amount of time. Actual results: The previous page has a noticeable pause (at least 1 sec on my old/slow PowerBook G4) before going back. Regression: This is a regression from shipping Safari 2.0.4 (419.3) on Mac OS X 10.4.8 (8L127). Tested with a local debug build of WebKit r19653 with the previously stated software. Notes: Does the internal Safari PLT suite test going backwards, too, or is that too non-deterministic? This is a recent regression (within the last week or so).
Comment 1 David Kilzer (:ddkilzer) 2007-02-16 07:14:02 PST
(In reply to comment #0) > Actual results: > > The previous page has a noticeable pause (at least 1 sec on my old/slow > PowerBook G4) before going back. But not THAT old/slow! :)
Comment 2 mitz 2007-02-16 08:08:38 PST
The delay is much shorter, but still noticeable, in a release build. I'm seeing the delay with the r19645 nightly but not with the r18516 nightly (didn't try to find the newest one that still doesn't have the delay). Creating a page that has even more links than the test page may make it easier to sample or Shark Safari.app and see what's causing the delay.
Comment 3 mitz 2007-02-16 11:25:47 PST
struct VectorComparer<false, T>::compare() in <http://trac.webkit.org/projects/webkit/changeset/19631> isn't very helpful :-)
Comment 4 Anders Carlsson 2007-02-16 11:27:26 PST
Committed revision 19661.
Comment 5 David Kilzer (:ddkilzer) 2007-06-24 12:04:48 PDT
Mass removal of NeedsRadar keyword from my bugs that have already been RESOLVED.