[webkit-reviews] review granted: [Bug 84490] PageGroupLoadDeferrer needs to take a ReasonForSuspension argument : [Attachment 138162] Patch v1 - Refactor to add a ReasonForSuspension argument

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Apr 20 14:18:26 PDT 2012


Anders Carlsson <andersca at apple.com> has granted Brady Eidson
<beidson at apple.com>'s request for review:
Bug 84490: PageGroupLoadDeferrer needs to take a ReasonForSuspension argument
https://bugs.webkit.org/show_bug.cgi?id=84490

Attachment 138162: Patch v1 - Refactor to add a ReasonForSuspension argument
https://bugs.webkit.org/attachment.cgi?id=138162&action=review

------- Additional Comments from Anders Carlsson <andersca at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=138162&action=review


> Source/WebCore/dom/Document.h:1486
> +    bool m_suspendedScheduledTasks;

m_suspendedScheduledTasks sounds like it would be a container for suspended
scheduled tasks. Maybe m_scheduledTasksAreSuspended or something that indicates
that it's a boolean?


More information about the webkit-reviews mailing list