Oilpan: eagerly finalize SVGFEImageElement as an ImageResourceClient.

Oilpan: eagerly finalize SVGFEImageElement as an ImageResourceClient.

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

[sof, 2015-06-23 06:33:47]

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

[sof, 2015-06-23 06:33:47]

please take a look.

[haraken, 2015-06-23 06:37:56]

Just a question, but would it be an option that we move ResourceClient to the
heap and mark it as EAGERLY_FINALIZE?

[sof, 2015-06-23 06:41:28]

On 2015/06/23 06:37:56, haraken wrote:
> Just a question, but would it be an option that we move ResourceClient to the
> heap and mark it as EAGERLY_FINALIZE?

You have non-heap objects that are clients, which makes this (and other Resource
changes) tricky. Like LayoutObject.

[haraken, 2015-06-23 06:42:54]

On 2015/06/23 06:41:28, sof wrote:
> On 2015/06/23 06:37:56, haraken wrote:
> > Just a question, but would it be an option that we move ResourceClient to
the
> > heap and mark it as EAGERLY_FINALIZE?
> 
> You have non-heap objects that are clients, which makes this (and other
Resource
> changes) tricky. Like LayoutObject.

Thanks, makes sense. LGTM.

[sof, 2015-06-23 09:43:12]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-06-23 09:43:27]

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

[commit-bot: I haz the power, 2015-06-23 09:46:26]

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