Summary: | REGRESSION(r17256-17272): Multiple history entries created | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Carson <dacarson> | ||||
Component: | History | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | beidson, mjs, mrowe | ||||
Priority: | P1 | Keywords: | HasReduction, InRadar, Regression | ||||
Version: | 420+ | ||||||
Hardware: | Mac | ||||||
OS: | OS X 10.4 | ||||||
URL: | http://cbs4boston.com/ | ||||||
Attachments: |
|
Description
David Carson
2006-12-27 14:20:42 PST
Theres been alot of loader and history work done lately... any chance you can regress this using the nightlys? Regress using the nightlys, meaning going back through the nightlys and seeing where the problem first appears The additional history entries come from the titles advertisements that are displayed in iframes. This regression would appear to be introduced between r17256 and r17282, but it's hard to narrow it down further as the URL in question crashes the nightly builds in that range. Created attachment 12659 [details]
Reduction
The ad-related JS on the page creates an iframe via document.write, then straight away sets its src property. It appears that setting the src property to the same value is triggering a fresh load which creates a history entry.
*** Bug 12650 has been marked as a duplicate of this bug. *** From duplicate: <rdar://problem/4921797> (In reply to comment #7) > The ad-related JS on the page creates an iframe via document.write, then > straight away sets its src property. It appears that setting the src property > to the same value is triggering a fresh load which creates a history entry. Setting the src property is always supposed to cause a reload. But that was buggy in older version of WebKit. It didn't trigger a load if the URL was the same. It's possible that we need to trigger a load yet somehow know not to put things in the back/forward list in this case. Using the reduction, I regressed this further with the nightlies to the 17256-17272 range This broke in http://trac.webkit.org/projects/webkit/changeset/17267 Geoff's seemingly simplistic change that was attempting to clean up loading code, but there's something subtle it tweaked re: user gesture versus non-user gesture here that I need to figure out. The reduction Mark posted way key - it was well understood and a fix was committed in r20813! Yay! |