Oilpan: Change the name of the finalization method that can be

Oilpan: Change the name of the finalization method that can be
overridden for explicit destructor dispatching.

The |finalize| name is too short and makes sense in other places
in the code. We rarely need to override finalizeGarbageCollectedObject
so it is fine that the method name is long.

Concretely, fileapi Stream has a finalize method that currently
gets called instead of the destructor. That leads to crashes because
the Stream destructor is not called and therefore the Stream is
no unregistered as a ContextLifecycleObserver and a call is made
on the already dead stream when the Document dies.

R=erik.corry@gmail.com, haraken@chromium.org, oilpan-reviews@chromium.org, tkent@chromium.org, zerny@chromium.org
BUG=352755
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169548

[haraken, 2014-03-19 12:26:05]

LGTM

[Mads Ager (chromium), 2014-03-19 12:37:59]

Jochen or Kent, could you stamp wtf/Vector.h change?

[jochen (gone - plz use gerrit), 2014-03-19 12:39:27]

lgtm

[Mads Ager (chromium), 2014-03-19 12:39:52]

On 2014/03/19 12:39:27, jochen wrote:
> lgtm

Thanks Jochen! :)

[Mads Ager (chromium), 2014-03-19 12:39:56]

The CQ bit was checked by ager@chromium.org

[commit-bot: I haz the power, 2014-03-19 12:39:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ager@chromium.org/199273005/1

[commit-bot: I haz the power, 2014-03-19 12:42:11]

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

[commit-bot: I haz the power, 2014-03-19 12:42:11]

Try jobs failed on following builders:
  tryserver.blink on win_blink_rel

[Mads Ager (chromium), 2014-03-19 12:46:50]

The CQ bit was checked by ager@chromium.org

[commit-bot: I haz the power, 2014-03-19 12:47:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ager@chromium.org/199273005/1

[commit-bot: I haz the power, 2014-03-19 13:45:39]

Change committed as 169548

[zerny-chromium, 2014-03-21 07:21:01]

QQ

https://codereview.chromium.org/199273005/diff/1/Source/wtf/Vector.h
File Source/wtf/Vector.h (right):

https://codereview.chromium.org/199273005/diff/1/Source/wtf/Vector.h#newcode612
Source/wtf/Vector.h:612: finalize();
Is there a particular reason finalize is not renamed here?

[Mads Ager (google), 2014-03-21 07:26:38]

https://codereview.chromium.org/199273005/diff/1/Source/wtf/Vector.h
File Source/wtf/Vector.h (right):

https://codereview.chromium.org/199273005/diff/1/Source/wtf/Vector.h#newcode612
Source/wtf/Vector.h:612: finalize();
On 2014/03/21 07:21:01, zerny-chromium wrote:
> Is there a particular reason finalize is not renamed here?

Yeah, finalize is also called in the non-oilpan build.