PingLoaderImpl should use ContextClient instead of using WeakMember

PingLoaderImpl should use ContextClient instead of using WeakMember

PingLoaderImpl is expecting frame() to be cleared when the frame is detached.
This means that it needs to be ContextClient.
Using WeakMember<LocalFrame> is wrong because frame() is not cleared until
the frame is gc-ed.

BUG=610176
Committed: https://crrev.com/01c3564cde9cbaa8705f128337f3bed44c2d6ae3
Cr-Commit-Position: refs/heads/master@{#440085}

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

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

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

PTAL

(This is a correct reland of https://codereview.chromium.org/2569983002/)

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

lgtm

[haraken, 2016-12-21 12:38:16]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-21 12:38:26]

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 13:54:42]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482323896723380,
"parent_rev": "59eee905aec4e872e9d7e41d7dd2e91ceea15c89", "commit_rev":
"9473bb39810e0f251549b920c08f00c51a5c692d"}

[commit-bot: I haz the power, 2016-12-21 13:55:09]

Description was changed from

==========
PingLoaderImpl should use ContextClient instead of using WeakMember

PingLoaderImpl is expecting frame() to be cleared when the frame is detached.
This means that it needs to be ContextClient.
Using WeakMember<LocalFrame> is wrong because frame() is not cleared until
the frame is gc-ed.

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

to

==========
PingLoaderImpl should use ContextClient instead of using WeakMember

PingLoaderImpl is expecting frame() to be cleared when the frame is detached.
This means that it needs to be ContextClient.
Using WeakMember<LocalFrame> is wrong because frame() is not cleared until
the frame is gc-ed.

BUG=610176

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

[commit-bot: I haz the power, 2016-12-21 13:55:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-21 13:57:36]

Description was changed from

==========
PingLoaderImpl should use ContextClient instead of using WeakMember

PingLoaderImpl is expecting frame() to be cleared when the frame is detached.
This means that it needs to be ContextClient.
Using WeakMember<LocalFrame> is wrong because frame() is not cleared until
the frame is gc-ed.

BUG=610176

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

to

==========
PingLoaderImpl should use ContextClient instead of using WeakMember

PingLoaderImpl is expecting frame() to be cleared when the frame is detached.
This means that it needs to be ContextClient.
Using WeakMember<LocalFrame> is wrong because frame() is not cleared until
the frame is gc-ed.

BUG=610176

Committed: https://crrev.com/01c3564cde9cbaa8705f128337f3bed44c2d6ae3
Cr-Commit-Position: refs/heads/master@{#440085}
==========

[commit-bot: I haz the power, 2016-12-21 13:57:37]

Patchset 2 (id:??) landed as
https://crrev.com/01c3564cde9cbaa8705f128337f3bed44c2d6ae3
Cr-Commit-Position: refs/heads/master@{#440085}