Remove unnecessary uses of GarbageCollectedFinalized<>.

Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

R=haraken
BUG=585328
Committed: https://crrev.com/e10d108bc9cabf82607b909f35fc541c313e2366
Cr-Commit-Position: refs/heads/master@{#390070}

Committed: https://crrev.com/187a2805642014aead68645a21dd3ac2c3c5cecb
Cr-Commit-Position: refs/heads/master@{#390165}

[sof, 2016-04-27 12:20:23]

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

[sof, 2016-04-27 12:20:24]

please take a look.

With https://codereview.chromium.org/1922913004/ in scope, clear out the
remaining avoidable uses of GarbageCollectedFinalized<>.

[haraken, 2016-04-27 12:32:44]

LGTM

[sof, 2016-04-27 13:20:44]

Description was changed from

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

R=
BUG=585328
==========

to

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

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

[sof, 2016-04-27 13:21:09]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2016-04-27 13:21:10]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/1929493002/#ps20001
(title: "add two leftover classes needing same treatment")

[commit-bot: I haz the power, 2016-04-27 13:21:24]

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

[commit-bot: I haz the power, 2016-04-27 14:45:21]

Description was changed from

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

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

to

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

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

[commit-bot: I haz the power, 2016-04-27 14:45:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-27 14:48:07]

Description was changed from

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

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

to

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

R=haraken
BUG=585328

Committed: https://crrev.com/e10d108bc9cabf82607b909f35fc541c313e2366
Cr-Commit-Position: refs/heads/master@{#390070}
==========

[commit-bot: I haz the power, 2016-04-27 14:48:09]

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

[Justin Donnelly, 2016-04-27 19:05:31]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1923993002/ by jdonnelly@chromium.org.

The reason for reverting is: Suspected for breaking
serviceworker-echo-referrer-from-default-document.html on the WebKit
{Linux,Win7} (dbg) bots..

[sof, 2016-04-27 20:06:54]

On 2016/04/27 19:05:31, Justin Donnelly wrote:
> A revert of this CL (patchset #2 id:20001) has been created in
> https://codereview.chromium.org/1923993002/ by mailto:jdonnelly@chromium.org.
> 
> The reason for reverting is: Suspected for breaking
> serviceworker-echo-referrer-from-default-document.html on the WebKit
> {Linux,Win7} (dbg) bots..

Relanding (via CQ); see https://codereview.chromium.org/1923993002/

[sof, 2016-04-27 20:07:14]

Description was changed from

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

R=haraken
BUG=585328

Committed: https://crrev.com/e10d108bc9cabf82607b909f35fc541c313e2366
Cr-Commit-Position: refs/heads/master@{#390070}
==========

to

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

R=haraken
BUG=585328

Committed: https://crrev.com/e10d108bc9cabf82607b909f35fc541c313e2366
Cr-Commit-Position: refs/heads/master@{#390070}
==========

[sof, 2016-04-27 20:07:24]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-04-27 20:07:59]

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

[commit-bot: I haz the power, 2016-04-27 20:13:44]

Description was changed from

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

R=haraken
BUG=585328

Committed: https://crrev.com/e10d108bc9cabf82607b909f35fc541c313e2366
Cr-Commit-Position: refs/heads/master@{#390070}
==========

to

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

R=haraken
BUG=585328

Committed: https://crrev.com/e10d108bc9cabf82607b909f35fc541c313e2366
Cr-Commit-Position: refs/heads/master@{#390070}
==========

[commit-bot: I haz the power, 2016-04-27 20:13:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-30 17:12:31]

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

[commit-bot: I haz the power, 2016-04-30 17:51:34]

Description was changed from

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

R=haraken
BUG=585328

Committed: https://crrev.com/e10d108bc9cabf82607b909f35fc541c313e2366
Cr-Commit-Position: refs/heads/master@{#390070}
==========

to

==========
Remove unnecessary uses of GarbageCollectedFinalized<>.

Classes which finalize nothing on their own should derive from
GarbageCollected<> instead.

(As identified by the warn-unneeded-finalizer=1 GC plugin option.)

R=haraken
BUG=585328

Committed: https://crrev.com/e10d108bc9cabf82607b909f35fc541c313e2366
Cr-Commit-Position: refs/heads/master@{#390070}

Committed: https://crrev.com/187a2805642014aead68645a21dd3ac2c3c5cecb
Cr-Commit-Position: refs/heads/master@{#390165}
==========