wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1037911: Downgrade assertion on GMP codec destruction since an edge case triggers it r=cpearce

Browse Source
This commit is contained in:
Randell Jesup 2014-07-13 02:06:58 -04:00
parent 8031891b11
commit 2c93f480c3
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
content/media/gmp/GMPParent.cpp
View File

@ -126,7 +126,8 @@ GMPParent::VideoDecoderDestroyed(GMPVideoDecoderParent* aDecoder)
{
MOZ_ASSERT(GMPThread() == NS_GetCurrentThread());
MOZ_ALWAYS_TRUE(mVideoDecoders.RemoveElement(aDecoder));
// If the constructor fails, we'll get called before it's added
NS_WARN_IF(!mVideoDecoders.RemoveElement(aDecoder));
// Recv__delete__ is on the stack, don't potentially destroy the top-level actor
// until after this has completed.
@ -139,7 +140,8 @@ GMPParent::VideoEncoderDestroyed(GMPVideoEncoderParent* aEncoder)
{
MOZ_ASSERT(GMPThread() == NS_GetCurrentThread());
MOZ_ALWAYS_TRUE(mVideoEncoders.RemoveElement(aEncoder));
// If the constructor fails, we'll get called before it's added
NS_WARN_IF(!mVideoEncoders.RemoveElement(aEncoder));
// Recv__delete__ is on the stack, don't potentially destroy the top-level actor
// until after this has completed.