[webkit-reviews] review requested: [Bug 53733] Timers can fire after a frame has been put into the page cache : [Attachment 124286] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jan 27 04:01:36 PST 2012


Allan Sandfeld Jensen <allan.jensen at nokia.com> has asked  for review:
Bug 53733: Timers can fire after a frame has been put into the page cache
https://bugs.webkit.org/show_bug.cgi?id=53733

Attachment 124286: Patch
https://bugs.webkit.org/attachment.cgi?id=124286&action=review

------- Additional Comments from Allan Sandfeld Jensen <allan.jensen at nokia.com>
Updated patch. 

Fixed comments.

Call didInstallActiveDOMObjects for AbstractWorker subclasses as well, even
though they don't use the suspend API, this makes the
didInstallActiveDOMObjects universally called from all descendants of
ActiveDOMObjects.

Moved expected result to qt only, because I do not have the ability to test all
platforms.


More information about the webkit-reviews mailing list