Bug 84950 – [GTK] Massive media tests failures since r115288

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Philippe Normand]

Lots of tests now timing out. It seems the GStreamer media player does something wrong which has bad consequences on the memory management of the media elements.

This works fine:

run-webkit-tests --run-singly --gtk media

But when the DRT instance is reused acrossed tests (default behavior) it triggers unexpected flakiness of the media suite.

[Mark Hahnenberg]

I believe this has to do with the fact that we only have a properly implemented GCActivityCallback on platforms which have CoreFoundation, meaning we are now not doing any collection for those platforms without CoreFoundation when we call GCController::garbageCollectSoon(). This is obviously bad. 

I'd imagine that this issue manifests itself in the media tests because of the fact that they're dealing with very large things that need to be collected pretty quickly when they die to prevent memory bloat.

[Mark Hahnenberg]

Created attachment 139040 [details]
Patch

[Philippe Normand]

I've just tested the patch locally and it fixes the media suite flakyness. Thanks Mark!

[WebKit Review Bot]

Comment on attachment 139040 [details]
Patch

Clearing flags on attachment: 139040

Committed r115383: <http://trac.webkit.org/changeset/115383>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Yong Li]

How about moving Timer/SharedTimer into WTF as is done for String?

[Mark Hahnenberg]

(In reply to comment #6)
> How about moving Timer/SharedTimer into WTF as is done for String?

Yes, that is what I'd like to do when I get around to it. If you'd like to do that, you're more than welcome! :-)