[Webkit-unassigned] [Bug 53733] Timers can fire after a frame has been put into the page cache

Mon Jan 23 12:45:02 PST 2012

https://bugs.webkit.org/show_bug.cgi?id=53733

--- Comment #9 from Allan Sandfeld <sandfeld at kde.org>  2012-01-23 12:45:02 PST ---
(In reply to comment #8)
> Like it or not your patch in 76063 *DOES* do two logically separate things.  It fixes a cross platform WebCore bug and it adds new WebKit2 API.
> 

I do not think it does two separate things. If this bug did not exist it would not have been an issue. It was pure accident I stumbled across this bug in the code comments. But for bugzilla clarity I wouldn't mind separating the patch. I do like to do at least manual tests, but this has been done, and the code is clear enough.

But please note that the last remaining reviewer comments I have against my patch (by Simon Fraser), are on the parts that would be a submitted here instead.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.