Revert of Remove unnecessary uses of GarbageCollectedFinalized<>.

Revert of Remove unnecessary uses of GarbageCollectedFinalized<>. (patchset #2 id:20001 of https://codereview.chromium.org/1929493002/ )

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

Original issue's description:
> 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}

TBR=oilpan-reviews@chromium.org,sigbjornf@opera.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=585328
Committed: https://crrev.com/09b39e6a88de47d374f2246d92092fe6222fd4dd
Cr-Commit-Position: refs/heads/master@{#390144}

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

Created Revert of Remove unnecessary uses of GarbageCollectedFinalized<>.

[commit-bot: I haz the power, 2016-04-27 19:05:50]

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

[commit-bot: I haz the power, 2016-04-27 19:08:38]

Committed patchset #1 (id:1)

[haraken, 2016-04-27 19:26:53]

LGTM

[sof, 2016-04-27 19:28:57]

On 2016/04/27 19:08:38, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1)

But isn't this https://bugs.chromium.org/p/chromium/issues/detail?id=607156 ?

[sof, 2016-04-27 19:43:25]

On 2016/04/27 19:28:57, sof wrote:
> On 2016/04/27 19:08:38, commit-bot: I haz the power wrote:
> > Committed patchset #1 (id:1)
> 
> But isn't this https://bugs.chromium.org/p/chromium/issues/detail?id=607156 ?

That is, it is flakily failing here on linux dbg


https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20(dbg)/...

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux__db...
 
which is outside the range of this CL. See also
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=htt...
for other and earlier failures.

I'll reland.

[Justin Donnelly, 2016-04-27 19:44:50]

On 2016/04/27 19:28:57, sof wrote:
> On 2016/04/27 19:08:38, commit-bot: I haz the power wrote:
> > Committed patchset #1 (id:1)
> 
> But isn't this https://bugs.chromium.org/p/chromium/issues/detail?id=607156 ?

Hmm, yeah, you're right. I was aware of that issue but even though it's the same
test failing I misread the logs. Sorry for the trouble.

[commit-bot: I haz the power, 2016-04-30 17:11:47]

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

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

Description was changed from

==========
Revert of Remove unnecessary uses of GarbageCollectedFinalized<>. (patchset #2
id:20001 of https://codereview.chromium.org/1929493002/ )

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

Original issue's description:
> 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}

TBR=oilpan-reviews@chromium.org,sigbjornf@opera.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=585328
==========

to

==========
Revert of Remove unnecessary uses of GarbageCollectedFinalized<>. (patchset #2
id:20001 of https://codereview.chromium.org/1929493002/ )

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

Original issue's description:
> 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}

TBR=oilpan-reviews@chromium.org,sigbjornf@opera.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=585328

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