Revert of Make AbstractWorker a ContextLifecycleObserver rather than an ActiveDOMObject

Revert of Make AbstractWorker a ContextLifecycleObserver rather than an ActiveDOMObject (patchset #3 id:40001 of https://codereview.chromium.org/1744973002/ )

Reason for revert:
I noticed that we're not ready to remove the m_stopped variable (because
LifecycleObserver does not yet have an ability to clear
LifecycleObserver::m_context when the context is destroyed). We need to fix the
LifecycleObserver before landing this fix.

Original issue's description:
> Make AbstractWorker a ContextLifecycleObserver rather than an ActiveDOMObject
>
> Per the plan described in https://groups.google.com/a/chromium.org/d/topic/blink-dev/UwRGJgLBElo/discussion,
> this CL reduces ActiveDOMObjects in the code base.
>
> - This CL replaces ServiceWorker::m_wasStopped with ContextLifecycleObserver::executionContext().
>
> - This CL removes an 'if(AbstractWorker::hasPendingActivity)' check because it always returns true.
>
> BUG=589507
>
> Committed: https://crrev.com/21ab87586c14afb2a22c91b05d932563cba9569d
> Cr-Commit-Position: refs/heads/master@{#378332}

TBR=kinuko@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=589507
Committed: https://crrev.com/99ccb8b5651202f197625695d87bd548ef94aeab
Cr-Commit-Position: refs/heads/master@{#378997}

[haraken, 2016-03-03 10:05:48]

Created Revert of Make AbstractWorker a ContextLifecycleObserver rather than an
ActiveDOMObject

[commit-bot: I haz the power, 2016-03-03 10:05:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1765493002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1765493002/1

[sof, 2016-03-03 10:06:53]

lgtm

[commit-bot: I haz the power, 2016-03-03 10:31:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1765493002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1765493002/1

[commit-bot: I haz the power, 2016-03-03 11:01:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1765493002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1765493002/1

[commit-bot: I haz the power, 2016-03-03 11:20:45]

Description was changed from

==========
Revert of Make AbstractWorker a ContextLifecycleObserver rather than an
ActiveDOMObject (patchset #3 id:40001 of
https://codereview.chromium.org/1744973002/ )

Reason for revert:
I noticed that we're not ready to remove the m_stopped variable (because
LifecycleObserver does not yet have an ability to clear
LifecycleObserver::m_context when the context is destroyed). We need to fix the
LifecycleObserver before landing this fix.

Original issue's description:
> Make AbstractWorker a ContextLifecycleObserver rather than an ActiveDOMObject
> 
> Per the plan described in
https://groups.google.com/a/chromium.org/d/topic/blink-dev/UwRGJgLBElo/discus...,
> this CL reduces ActiveDOMObjects in the code base.
> 
> - This CL replaces ServiceWorker::m_wasStopped with
ContextLifecycleObserver::executionContext().
> 
> - This CL removes an 'if(AbstractWorker::hasPendingActivity)' check because it
always returns true.
> 
> BUG=589507
> 
> Committed: https://crrev.com/21ab87586c14afb2a22c91b05d932563cba9569d
> Cr-Commit-Position: refs/heads/master@{#378332}

TBR=kinuko@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=589507
==========

to

==========
Revert of Make AbstractWorker a ContextLifecycleObserver rather than an
ActiveDOMObject (patchset #3 id:40001 of
https://codereview.chromium.org/1744973002/ )

Reason for revert:
I noticed that we're not ready to remove the m_stopped variable (because
LifecycleObserver does not yet have an ability to clear
LifecycleObserver::m_context when the context is destroyed). We need to fix the
LifecycleObserver before landing this fix.

Original issue's description:
> Make AbstractWorker a ContextLifecycleObserver rather than an ActiveDOMObject
> 
> Per the plan described in
https://groups.google.com/a/chromium.org/d/topic/blink-dev/UwRGJgLBElo/discus...,
> this CL reduces ActiveDOMObjects in the code base.
> 
> - This CL replaces ServiceWorker::m_wasStopped with
ContextLifecycleObserver::executionContext().
> 
> - This CL removes an 'if(AbstractWorker::hasPendingActivity)' check because it
always returns true.
> 
> BUG=589507
> 
> Committed: https://crrev.com/21ab87586c14afb2a22c91b05d932563cba9569d
> Cr-Commit-Position: refs/heads/master@{#378332}

TBR=kinuko@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=589507
==========

[commit-bot: I haz the power, 2016-03-03 11:20:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-03 11:22:17]

Description was changed from

==========
Revert of Make AbstractWorker a ContextLifecycleObserver rather than an
ActiveDOMObject (patchset #3 id:40001 of
https://codereview.chromium.org/1744973002/ )

Reason for revert:
I noticed that we're not ready to remove the m_stopped variable (because
LifecycleObserver does not yet have an ability to clear
LifecycleObserver::m_context when the context is destroyed). We need to fix the
LifecycleObserver before landing this fix.

Original issue's description:
> Make AbstractWorker a ContextLifecycleObserver rather than an ActiveDOMObject
> 
> Per the plan described in
https://groups.google.com/a/chromium.org/d/topic/blink-dev/UwRGJgLBElo/discus...,
> this CL reduces ActiveDOMObjects in the code base.
> 
> - This CL replaces ServiceWorker::m_wasStopped with
ContextLifecycleObserver::executionContext().
> 
> - This CL removes an 'if(AbstractWorker::hasPendingActivity)' check because it
always returns true.
> 
> BUG=589507
> 
> Committed: https://crrev.com/21ab87586c14afb2a22c91b05d932563cba9569d
> Cr-Commit-Position: refs/heads/master@{#378332}

TBR=kinuko@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=589507
==========

to

==========
Revert of Make AbstractWorker a ContextLifecycleObserver rather than an
ActiveDOMObject (patchset #3 id:40001 of
https://codereview.chromium.org/1744973002/ )

Reason for revert:
I noticed that we're not ready to remove the m_stopped variable (because
LifecycleObserver does not yet have an ability to clear
LifecycleObserver::m_context when the context is destroyed). We need to fix the
LifecycleObserver before landing this fix.

Original issue's description:
> Make AbstractWorker a ContextLifecycleObserver rather than an ActiveDOMObject
>
> Per the plan described in
https://groups.google.com/a/chromium.org/d/topic/blink-dev/UwRGJgLBElo/discus...,
> this CL reduces ActiveDOMObjects in the code base.
>
> - This CL replaces ServiceWorker::m_wasStopped with
ContextLifecycleObserver::executionContext().
>
> - This CL removes an 'if(AbstractWorker::hasPendingActivity)' check because it
always returns true.
>
> BUG=589507
>
> Committed: https://crrev.com/21ab87586c14afb2a22c91b05d932563cba9569d
> Cr-Commit-Position: refs/heads/master@{#378332}

TBR=kinuko@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=589507

Committed: https://crrev.com/99ccb8b5651202f197625695d87bd548ef94aeab
Cr-Commit-Position: refs/heads/master@{#378997}
==========

[commit-bot: I haz the power, 2016-03-03 11:22:18]

Patchset 1 (id:??) landed as
https://crrev.com/99ccb8b5651202f197625695d87bd548ef94aeab
Cr-Commit-Position: refs/heads/master@{#378997}

[Finnur, 2016-03-03 15:16:04]

Also, it seems a lot of the service worker tests went green as a result of this
revert.
See crbug.com/591596 for details.

[kinuko, 2016-03-07 02:21:53]

lgtm.