[Webkit-unassigned] [Bug 47020] Fixing crash when audio media player is destructed

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Oct 1 18:50:47 PDT 2010


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


James Robinson <jamesr at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #69533|review?                     |review+
               Flag|                            |




--- Comment #10 from James Robinson <jamesr at chromium.org>  2010-10-01 18:50:47 PST ---
(From update of attachment 69533)
View in context: https://bugs.webkit.org/attachment.cgi?id=69533&action=review

R=me

> WebCore/platform/graphics/chromium/VideoLayerChromium.cpp:190
> +    if (!layerRenderer())
> +        return;
> +

If we expect this never to be the case, it's probably better to add an ASSERT() instead and watch for issues in the field.  We don't want to paper over real bugs if we don't understand how they might be happening.

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



More information about the webkit-unassigned mailing list