Make InterpolationRecord GarbageCollectedFinalized

Make InterpolationRecord GarbageCollectedFinalized

Since InterpolationRecord has a RefPtr, it should be GarbageCollectedFinalized.

BUG=340522
TBR=ager

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

[haraken, 2014-03-25 12:15:19]

Committed patchset #2 manually as r169935 (presubmit successful).

[Mads Ager (chromium), 2014-03-25 12:16:20]

LGTM, thanks

[wibling-chromium, 2014-03-25 12:35:48]

lgtm

[zerny-chromium, 2014-03-25 12:58:54]

This should have been caught by the plugin but does not appear to be.
Investigating.


On Tue, Mar 25, 2014 at 1:35 PM, <wibling@chromium.org> wrote:

> lgtm
>
>
>
> https://codereview.chromium.org/208283005/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[haraken, 2014-03-25 13:23:14]

On 2014/03/25 12:58:54, zerny-chromium wrote:
> This should have been caught by the plugin but does not appear to be.
> Investigating.

Sorry, I didn't run the plugin :) (Now I have an error when running the plugin
probably due to my local environment setting. Investigating.)

[zerny-chromium, 2014-03-25 13:26:42]

It might be the error I've fixed in:
https://codereview.chromium.org/204543005/

I'm not sure the plugin actually caches the error in this CL. Maybe due to
InterpolationRecord being an inner class. I'll take a look at this.


On Tue, Mar 25, 2014 at 2:23 PM, <haraken@chromium.org> wrote:

> On 2014/03/25 12:58:54, zerny-chromium wrote:
>
>> This should have been caught by the plugin but does not appear to be.
>> Investigating.
>>
>
> Sorry, I didn't run the plugin :) (Now I have an error when running the
> plugin
> probably due to my local environment setting. Investigating.)
>
>
> https://codereview.chromium.org/208283005/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.