Revert of Remove ContextLifecycleObserver from ScriptPromisePropertyBase

Revert of Remove ContextLifecycleObserver from ScriptPromisePropertyBase (patchset #1 id:1 of https://codereview.chromium.org/2555873004/ )

Reason for revert:
This caused us to start failing the following tests on WebKit Linux Trusty Leak (verified by local bisect):
virtual/service-worker-navigation-preload/http/tests/serviceworker/ready.html
fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html
fast/dom/Window/property-access-on-cached-properties-after-frame-removed.html
fast/dom/Window/property-access-on-cached-window-after-frame-removed.html
fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html
virtual/mojo-loading/http/tests/serviceworker/ready.html
fast/dom/Window/property-access-on-cached-properties-after-frame-navigated.html
fast/dom/Window/property-access-on-cached-window-after-frame-navigated.html
virtual/disable-mojo-service-worker/http/tests/serviceworker/ready.html
imported/wpt/service-workers/service-worker/ready.https.html
http/tests/serviceworker/ready.html

First failure:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak/builds/559

Original issue's description:
> Remove ContextLifecycleObserver from ScriptPromisePropertyBase
>
> ScriptPromisePropertyBase doesn't need to inherit from ContextLifecycleObserver.
> It can just hold a strong member to ExecutionContext.
>
> BUG=610176
>
> Committed: https://crrev.com/da5aebc18deaf9e1f7154df19426abf91e877913
> Cr-Commit-Position: refs/heads/master@{#436950}

TBR=sigbjornf@opera.com,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=610176
Committed: https://crrev.com/4383bedae77f70cdb2b22ad5dd684385f835d501
Cr-Commit-Position: refs/heads/master@{#437076}

[flackr, 2016-12-07 22:05:05]

The CQ bit was checked by flackr@chromium.org

[flackr, 2016-12-07 22:05:06]

Created Revert of Remove ContextLifecycleObserver from ScriptPromisePropertyBase

[commit-bot: I haz the power, 2016-12-07 22:05:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-07 22:06:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481148305819820, "parent_rev":
"156e547527ce102e9357803f450f10e359110be1", "commit_rev":
"1965de252f0428b2250a85202dc1791de0e4661e"}

[commit-bot: I haz the power, 2016-12-07 22:06:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-07 22:10:18]

Description was changed from

==========
Revert of Remove ContextLifecycleObserver from ScriptPromisePropertyBase
(patchset #1 id:1 of https://codereview.chromium.org/2555873004/ )

Reason for revert:
This caused us to start failing the following tests on WebKit Linux Trusty Leak
(verified by local bisect):
virtual/service-worker-navigation-preload/http/tests/serviceworker/ready.html
fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html
fast/dom/Window/property-access-on-cached-properties-after-frame-removed.html
fast/dom/Window/property-access-on-cached-window-after-frame-removed.html
fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html
virtual/mojo-loading/http/tests/serviceworker/ready.html
fast/dom/Window/property-access-on-cached-properties-after-frame-navigated.html
fast/dom/Window/property-access-on-cached-window-after-frame-navigated.html
virtual/disable-mojo-service-worker/http/tests/serviceworker/ready.html
imported/wpt/service-workers/service-worker/ready.https.html
http/tests/serviceworker/ready.html

First failure:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty...

Original issue's description:
> Remove ContextLifecycleObserver from ScriptPromisePropertyBase
> 
> ScriptPromisePropertyBase doesn't need to inherit from
ContextLifecycleObserver.
> It can just hold a strong member to ExecutionContext.
> 
> BUG=610176
> 
> Committed: https://crrev.com/da5aebc18deaf9e1f7154df19426abf91e877913
> Cr-Commit-Position: refs/heads/master@{#436950}

TBR=sigbjornf@opera.com,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=610176
==========

to

==========
Revert of Remove ContextLifecycleObserver from ScriptPromisePropertyBase
(patchset #1 id:1 of https://codereview.chromium.org/2555873004/ )

Reason for revert:
This caused us to start failing the following tests on WebKit Linux Trusty Leak
(verified by local bisect):
virtual/service-worker-navigation-preload/http/tests/serviceworker/ready.html
fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html
fast/dom/Window/property-access-on-cached-properties-after-frame-removed.html
fast/dom/Window/property-access-on-cached-window-after-frame-removed.html
fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html
virtual/mojo-loading/http/tests/serviceworker/ready.html
fast/dom/Window/property-access-on-cached-properties-after-frame-navigated.html
fast/dom/Window/property-access-on-cached-window-after-frame-navigated.html
virtual/disable-mojo-service-worker/http/tests/serviceworker/ready.html
imported/wpt/service-workers/service-worker/ready.https.html
http/tests/serviceworker/ready.html

First failure:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty...

Original issue's description:
> Remove ContextLifecycleObserver from ScriptPromisePropertyBase
>
> ScriptPromisePropertyBase doesn't need to inherit from
ContextLifecycleObserver.
> It can just hold a strong member to ExecutionContext.
>
> BUG=610176
>
> Committed: https://crrev.com/da5aebc18deaf9e1f7154df19426abf91e877913
> Cr-Commit-Position: refs/heads/master@{#436950}

TBR=sigbjornf@opera.com,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=610176

Committed: https://crrev.com/4383bedae77f70cdb2b22ad5dd684385f835d501
Cr-Commit-Position: refs/heads/master@{#437076}
==========

[commit-bot: I haz the power, 2016-12-07 22:10:18]

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

[haraken, 2016-12-08 01:06:29]

Thanks, LGTM to revert.