[Webkit-unassigned] [Bug 47432] No longer ASSERT for LayerRenderer in VideoLayerChromium destructor

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=47432





--- Comment #17 from Vangelis Kokkevis <vangelis at chromium.org>  2010-10-13 12:42:42 PST ---
(In reply to comment #15)
> VideoLayerChromium shutdown has been the top crasher for the last few weeks now and we've been through several iterations of fixes but are still finding issues.  I think it's really high priority to get more test coverage here or we'll keep running in to more issues.

Agreed on adding tests for this type of situation.  Assuming that the failure is on devices where we fail to create a GL context I think the proper testing needs to happen at a different level and that's why I believe it's outside the scope of this CL. I don't have any good ideas on how, from JS only, to trigger a failure to create a GL context.

We could delay checking this in until we adjust our testing infrastructure to simulate a GL context creation failure and run through all our tests as I suggested in the chromium bug but I find this to be counter productive. This failure could be masking other video issues which we won't get to find until this particular crash is resolved.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.