ScriptPromisePropertyBase should use ContextClient instead of ContextLifecycleObserver

ScriptPromisePropertyBase should use ContextClient instead of ContextLifecycleObserver

ScriptPromisePropertyBase doesn't need to observe contextDestroyed.
So ContextClient is enough and more efficient.

BUG=610176
Committed: https://crrev.com/28a790f8946476da715d9ba72406f1dce462906b
Cr-Commit-Position: refs/heads/master@{#440087}

[haraken, 2016-12-21 12:28:28]

haraken@chromium.org changed reviewers:
+ sigbjornf@opera.com

[haraken, 2016-12-21 12:28:28]

PTAL

(Essentially this is a reland of https://codereview.chromium.org/2555873004/ in
a different style.)

[sof, 2016-12-21 12:30:50]

lgtm

The WeakMember<> preventing the leaks of the previous attempt, i hope.

[haraken, 2016-12-21 12:35:34]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-21 12:35:43]

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-21 14:03:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482323734111020, "parent_rev":
"5c51ebe6d27555663272c29d1163661c089244f4", "commit_rev":
"962caa4f6d46534190462502148dd2417e7ff1d3"}

[commit-bot: I haz the power, 2016-12-21 14:03:56]

Description was changed from

==========
ScriptPromisePropertyBase should use ContextClient instead of
ContextLifecycleObserver

ScriptPromisePropertyBase doesn't need to observe contextDestroyed.
So ContextClient is enough and more efficient.

BUG=610176
==========

to

==========
ScriptPromisePropertyBase should use ContextClient instead of
ContextLifecycleObserver

ScriptPromisePropertyBase doesn't need to observe contextDestroyed.
So ContextClient is enough and more efficient.

BUG=610176

Review-Url: https://codereview.chromium.org/2587323004
==========

[commit-bot: I haz the power, 2016-12-21 14:03:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-21 14:06:29]

Description was changed from

==========
ScriptPromisePropertyBase should use ContextClient instead of
ContextLifecycleObserver

ScriptPromisePropertyBase doesn't need to observe contextDestroyed.
So ContextClient is enough and more efficient.

BUG=610176

Review-Url: https://codereview.chromium.org/2587323004
==========

to

==========
ScriptPromisePropertyBase should use ContextClient instead of
ContextLifecycleObserver

ScriptPromisePropertyBase doesn't need to observe contextDestroyed.
So ContextClient is enough and more efficient.

BUG=610176

Committed: https://crrev.com/28a790f8946476da715d9ba72406f1dce462906b
Cr-Commit-Position: refs/heads/master@{#440087}
==========

[commit-bot: I haz the power, 2016-12-21 14:06:30]

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