Oilpan: correctly define transition macro for eager finalization.

Oilpan: correctly define transition macro for eager finalization.

EAGERLY_FINALIZE_WILL_BE_REMOVED() needs to provide eager finalization
outside of ENABLE(OILPAN), but only if lazy sweeping is enabled.

R=haraken
BUG=420522
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=196848

[sof, 2015-06-09 16:35:15]

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

[sof, 2015-06-09 16:35:16]

please take a look.

To avoid conflicts this needs to wait on
https://codereview.chromium.org/1172603002/

[sof, 2015-06-09 16:37:20]

The non-Oilpan fuzzer build will no doubt trip up on the lack of eager
finalization for ContextLifecycleObserver and DOMWindowProperty.

[haraken, 2015-06-09 23:27:51]

LGTM

[sof, 2015-06-10 07:52:37]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2015-06-10 07:52:38]

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/1170273002/#ps20001
(title: "rebased")

[commit-bot: I haz the power, 2015-06-10 07:52:49]

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

[commit-bot: I haz the power, 2015-06-10 09:15:25]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196848

[vabr (Chromium), 2015-06-11 11:47:38]

I think you might have got the bug number wrong. :)
Cheers,
Vaclav

[sof, 2015-06-11 11:52:05]

On 2015/06/11 11:47:38, vabr (Chromium) wrote:
> I think you might have got the bug number wrong. :)
> Cheers,
> Vaclav

Oops, sorry. 340522 and 420515 are two common Oilpan issues, I wanted the latter
here, but somehow managed to turn that into 420522.