Bug 117522 – Make sure we aren't throttling plugin timers during initialisation and destruction

• Component: New Bugs
• Status: RESOLVED FIXED

[Oliver Hunt]

Make sure we aren't throttling plugin timers during initialisation and destruction

[Oliver Hunt]

Created attachment 204362 [details]
Patch

[kov's GTK+ EWS bot]

Comment on attachment 204362 [details]
Patch

Attachment 204362 [details] did not pass gtk-wk2-ews (gtk-wk2):
Output: http://webkit-queues.appspot.com/results/789464

[Oliver Hunt]

radar://14103814

[Sam Weinig]

Comment on attachment 204362 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=204362&action=review

> Source/WebKit2/Shared/ActivityAssertion.cpp:28
> +#include "config.h"
> +
> +#include "ActivityAssertion.h"

You should not have newline here.

> Source/WebKit2/Shared/ActivityAssertion.h:36
> +class ChildProcess;
> +class ActivityAssertion {

You should have a newline here.

> Source/WebKit2/Shared/ActivityAssertion.h:39
> +    WTF_MAKE_NONCOPYABLE(ActivityAssertion);
> +
> +public:

You should not have a newline here.

> Source/WebKit2/Shared/ActivityAssertion.h:43
> +    static PassOwnPtr<ActivityAssertion> create(ChildProcess* process)
> +    {
> +        return adoptPtr(new ActivityAssertion(process));
> +    }

If this is only ever used on the stack, is there any reason to make this use OwnPtr/PassOwnPtr?

> Source/WebKit2/Shared/ActivityAssertion.h:50
> +    ActivityAssertion(ChildProcess*);
> +    ChildProcess* m_process;
> +};

You should have a newline here.

> Source/WebKit2/Shared/ChildProcess.h:82
> +    PassOwnPtr<ActivityAssertion> takeActivityAssertion()
> +    {
> +        return ActivityAssertion::create(this);
> +    }
> +    
> +

Extra newline.

[Oliver Hunt]

Created attachment 204374 [details]
Patch

[Darin Adler]

Comment on attachment 204374 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=204374&action=review

> Source/WebKit2/Shared/ActivityAssertion.h:31
> +#include <wtf/OwnPtr.h>
> +#include <wtf/PassOwnPtr.h>

Don’t need these any more.

[Sam Weinig]

Comment on attachment 204374 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=204374&action=review

> Source/WebKit2/PluginProcess/WebProcessConnection.cpp:232
> +    // Ensure we don't clamp any timers during initialisation

"initialisation" is spelled "initialization" in WebKit.

> Source/WebKit2/Shared/ActivityAssertion.h:46
> +private:
> +    ChildProcess& m_process;
> +
> +};

Extra newline.

[Oliver Hunt]

Committed r151480: <http://trac.webkit.org/changeset/151480>