Hide PingLoader lifetime implementation detail from outside view.

Hide PingLoader lifetime implementation detail from outside view.

The self-sustaining nature of ping loader objects while the request is
in-flight is an internal implementation detail. Reflect that by having
the class type derive from just GarbageCollectedFinalized> and instead
internally manually manage a SelfKeepAlive<> reference.

R=haraken
BUG=

Committed: https://crrev.com/4fdc302b6281499d53be92dcf151c91d51362235
Cr-Commit-Position: refs/heads/master@{#384810}

[sof, 2016-04-02 15:28:10]

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

[sof, 2016-04-02 15:28:11]

please take a look.

Moving one step closer to retiring RefCountedGarbageCollected<>..

[haraken, 2016-04-02 16:22:21]

LGTM

[sof, 2016-04-02 16:34:42]

Description was changed from

==========
Hide PingLoader lifetime implementation detail from outside view.

The self-sustaining nature of ping loader objects while the request is
in-flight is an internal implementation detail. Reflect that by having
the class type derive from just GarbageCollectedFinalized> and instead
internally manually manage a SelfKeepAlive<> reference.

R=
BUG=
==========

to

==========
Hide PingLoader lifetime implementation detail from outside view.

The self-sustaining nature of ping loader objects while the request is
in-flight is an internal implementation detail. Reflect that by having
the class type derive from just GarbageCollectedFinalized> and instead
internally manually manage a SelfKeepAlive<> reference.

R=haraken
BUG=
==========

[sof, 2016-04-02 16:34:49]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-04-02 16:34:58]

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

[commit-bot: I haz the power, 2016-04-02 17:39:19]

Description was changed from

==========
Hide PingLoader lifetime implementation detail from outside view.

The self-sustaining nature of ping loader objects while the request is
in-flight is an internal implementation detail. Reflect that by having
the class type derive from just GarbageCollectedFinalized> and instead
internally manually manage a SelfKeepAlive<> reference.

R=haraken
BUG=
==========

to

==========
Hide PingLoader lifetime implementation detail from outside view.

The self-sustaining nature of ping loader objects while the request is
in-flight is an internal implementation detail. Reflect that by having
the class type derive from just GarbageCollectedFinalized> and instead
internally manually manage a SelfKeepAlive<> reference.

R=haraken
BUG=
==========

[commit-bot: I haz the power, 2016-04-02 17:39:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-02 17:40:52]

Description was changed from

==========
Hide PingLoader lifetime implementation detail from outside view.

The self-sustaining nature of ping loader objects while the request is
in-flight is an internal implementation detail. Reflect that by having
the class type derive from just GarbageCollectedFinalized> and instead
internally manually manage a SelfKeepAlive<> reference.

R=haraken
BUG=
==========

to

==========
Hide PingLoader lifetime implementation detail from outside view.

The self-sustaining nature of ping loader objects while the request is
in-flight is an internal implementation detail. Reflect that by having
the class type derive from just GarbageCollectedFinalized> and instead
internally manually manage a SelfKeepAlive<> reference.

R=haraken
BUG=

Committed: https://crrev.com/4fdc302b6281499d53be92dcf151c91d51362235
Cr-Commit-Position: refs/heads/master@{#384810}
==========

[commit-bot: I haz the power, 2016-04-02 17:40:53]

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