RESOLVED FIXED 18560
Frame::clearTimers() calls suspendMarquees() on the wrong document 
https://bugs.webkit.org/show_bug.cgi?id=18560
Summary Frame::clearTimers() calls suspendMarquees() on the wrong document
Dean Jackson
Reported 2008-04-17 16:19:08 PDT
To reproduce: 1. Open a Safari window. 2. Drag 5-6 HTML files into the window, roughly one per second (to fill up the page cache). 3. Quickly drag the attached marquee test into the window. Wait for a few seconds. Note that the marquee stops. What's happening is that the page cache timer is firing, and calling Frame::clearTimers(). That method is getting a document via view->frame()->document(), but that's the currently loaded document, not the one in the page cache.
Attachments
patch for clearing timers (2.07 KB, patch)
2008-04-17 16:22 PDT, Dean Jackson 		hyatt: review+
DetailsFormatted DiffDiff
testcase for clearing timers (848 bytes, text/html)
2008-04-17 16:23 PDT, Dean Jackson 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Dean Jackson
Comment 1 2008-04-17 16:22:39 PDT
Created attachment 20646 [details] patch for clearing timers Patch is extracted from git, not svn
Dean Jackson
Comment 2 2008-04-17 16:23:17 PDT
Created attachment 20650 [details] testcase for clearing timers
Dave Hyatt
Comment 3 2008-04-17 17:46:14 PDT
Comment on attachment 20646 [details] patch for clearing timers r=me
Dean Jackson
Comment 4 2008-04-18 13:18:00 PDT
fixed in r32223
Simon Fraser (smfr)
Comment 5 2008-08-07 11:20:40 PDT
*** Bug 16995 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.