Oilpan: fix build after r194911.

Oilpan: fix build after r194911.

TBR=oilpan-reviews
BUG=484004
NOTRY=true

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194921

[sof, 2015-05-05 13:12:28]

please take a look.

A Persistent<Document> is unavoidable for StyleFetchedImage, as the use counter
has to be reported with respect to it.

Do clear out the reference upon 'finish' being notified though, as otherwise
this will create a leak.

[sof, 2015-05-05 13:23:03]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-05-05 13:23:19]

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

[commit-bot: I haz the power, 2015-05-05 13:26:57]

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

[haraken, 2015-05-05 15:57:57]

LGTM

> A Persistent<Document> is unavoidable for StyleFetchedImage, as the use
counter
> has to be reported with respect to it.

Would it be possible to move StyleFetchedImage to the heap (and avoid the
Persistent handle)?

[sof, 2015-05-05 17:33:53]

On 2015/05/05 15:57:57, haraken wrote:
> LGTM
> 
> > A Persistent<Document> is unavoidable for StyleFetchedImage, as the use
> counter
> > has to be reported with respect to it.
> 
> Would it be possible to move StyleFetchedImage to the heap (and avoid the
> Persistent handle)?

I don't see any immediate problems with doing that, but it would entail moving
StyleImage & its derived classes.