Bug 84490 - PageGroupLoadDeferrer needs to take a ReasonForSuspension argument
: PageGroupLoadDeferrer needs to take a ReasonForSuspension argument
Status: RESOLVED FIXED
: WebKit
Page Loading
: 528+ (Nightly build)
: All All
: P2 Normal
Assigned To:
:
:
:
: 84488
  Show dependency treegraph
 
Reported: 2012-04-20 13:50 PST by
Modified: 2012-04-20 14:29 PST (History)


Attachments
Patch v1 - Refactor to add a ReasonForSuspension argument (13.46 KB, patch)
2012-04-20 14:15 PST, Brady Eidson
andersca: review+
Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2012-04-20 13:50:18 PST
PageGroupLoadDeferrer needs to take a ReasonForSuspension argument

There's a long standing comment:
                // NOTE: if PageGroupLoadDeferrer is ever used for tasks other than showing a modal window or sheet,
                // the constructor will need to take a ActiveDOMObject::ReasonForSuspension.

As we resolve https://bugs.webkit.org/show_bug.cgi?id=84488 that comment is no longer a suggestion towards some vague future.

Also relevant to radar <rdar://problem/10484294>
------- Comment #1 From 2012-04-20 14:15:11 PST -------
Created an attachment (id=138162) [details]
Patch v1 - Refactor to add a ReasonForSuspension argument
------- Comment #2 From 2012-04-20 14:18:27 PST -------
(From update of attachment 138162 [details])
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?
------- Comment #3 From 2012-04-20 14:19:03 PST -------
(In reply to comment #2)
> (From update of attachment 138162 [details] [details])
> 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?

Sounds good.  Thanks!
------- Comment #4 From 2012-04-20 14:29:31 PST -------
http://trac.webkit.org/changeset/114782