Remove unnecessary eager finalization of PingLoaders.

Remove unnecessary eager finalization of PingLoaders.

A PingLoader instance controls its own lifetime, releasing its
self-keepalive reference upon completion of the load request. At that
time it also lets go of its loader object (and stops the timer), hence
there's no need to eagerly finalize this object -- it serves no real
purpose.

R=
BUG=

Committed: https://crrev.com/09d1e98f310cb1725bd97fa556fcb7298613e78a
Cr-Commit-Position: refs/heads/master@{#408417}

[sof, 2016-07-28 15:04:39]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-28 15:04:53]

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

[sof, 2016-07-28 15:06:09]

sigbjornf@opera.com changed reviewers:
+ oilpan-reviews@chromium.org

[sof, 2016-07-28 15:06:10]

please take a look.

[haraken, 2016-07-28 15:06:57]

LGTM

[commit-bot: I haz the power, 2016-07-28 15:09:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-28 15:09:36]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[sof, 2016-07-28 15:42:41]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-07-28 15:42:53]

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-07-28 17:29:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-28 17:32:28]

Description was changed from

==========
Remove unnecessary eager finalization of PingLoaders.

A PingLoader instance controls its own lifetime, releasing its
self-keepalive reference upon completion of the load request. At that
time it also lets go of its loader object (and stops the timer), hence
there's no need to eagerly finalize this object -- it serves no real
purpose.

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

to

==========
Remove unnecessary eager finalization of PingLoaders.

A PingLoader instance controls its own lifetime, releasing its
self-keepalive reference upon completion of the load request. At that
time it also lets go of its loader object (and stops the timer), hence
there's no need to eagerly finalize this object -- it serves no real
purpose.

R=
BUG=

Committed: https://crrev.com/09d1e98f310cb1725bd97fa556fcb7298613e78a
Cr-Commit-Position: refs/heads/master@{#408417}
==========

[commit-bot: I haz the power, 2016-07-28 17:32:29]

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