[SVG] call GCController.collectAll on SVG leak tests to address Oilpan flakiness.

[SVG] call GCController.collectAll on SVG leak tests to address Oilpan flakiness.

After r169941, the SVG leak tests reports failures, but delta is <0, so number of live nodes is not increasing. This CL loosen the check.

Also removes FIXME comments in the tests.

BUG=340522
NOTRY=true

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

[kouhei (in TOK), 2014-03-26 03:19:24]

+oilpan-reviews

[haraken, 2014-03-26 03:39:03]

It's mysterious that the delta becomes negative in the oilpan world. I might
want to understand what's going on there.

https://codereview.chromium.org/211933007/diff/1/LayoutTests/svg/animations/s...
File LayoutTests/svg/animations/smil-leak-element-instances.svg (right):

https://codereview.chromium.org/211933007/diff/1/LayoutTests/svg/animations/s...
LayoutTests/svg/animations/smil-leak-element-instances.svg:39:
GCController.collect();

What happens if we call GCController.collect() more times?

https://codereview.chromium.org/211933007/diff/1/LayoutTests/svg/animations/s...
LayoutTests/svg/animations/smil-leak-element-instances.svg:45:
GCController.collect();

Ditto.

[haraken, 2014-03-26 03:44:14]

I remember I implemented GCController.collectAll() which collects all garbage.
What happens if you use it?

[kouhei (in TOK), 2014-03-26 03:51:39]

On 2014/03/26 03:44:14, haraken wrote:
> I remember I implemented GCController.collectAll() which collects all garbage.
> What happens if you use it?

Done.

[haraken, 2014-03-26 03:55:38]

On 2014/03/26 03:51:39, kouhei wrote:
> On 2014/03/26 03:44:14, haraken wrote:
> > I remember I implemented GCController.collectAll() which collects all
garbage.
> > What happens if you use it?
> 
> Done.

Thanks, let's land this CL and see if the flakiness is gone. LGTM.

[kouhei (in TOK), 2014-03-26 03:55:56]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2014-03-26 03:55:57]

The CQ bit was unchecked by kouhei@chromium.org

[kouhei (in TOK), 2014-03-26 03:56:06]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2014-03-26 03:56:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kouhei@chromium.org/211933007/90001

[commit-bot: I haz the power, 2014-03-26 03:56:43]

Change committed as 170010