Bug 1116358 - Directly call Release() on |this| when closing a GMP encoder/decoder proxies; r=cpearce

This is needed in order to avoid calling Release() on a smart pointer.
This commit is contained in:
Ehsan Akhgari 2014-12-25 16:21:34 -05:00
Родитель 94821f46e8
Коммит de3498d101
3 изменённых файлов: 3 добавлений и 3 удалений

Просмотреть файл

@ -149,7 +149,7 @@ GMPAudioDecoderParent::Close()
// In case this is the last reference // In case this is the last reference
nsRefPtr<GMPAudioDecoderParent> kungfudeathgrip(this); nsRefPtr<GMPAudioDecoderParent> kungfudeathgrip(this);
NS_RELEASE(kungfudeathgrip); Release();
Shutdown(); Shutdown();
return NS_OK; return NS_OK;

Просмотреть файл

@ -76,7 +76,7 @@ GMPVideoDecoderParent::Close()
// In case this is the last reference // In case this is the last reference
nsRefPtr<GMPVideoDecoderParent> kungfudeathgrip(this); nsRefPtr<GMPVideoDecoderParent> kungfudeathgrip(this);
NS_RELEASE(kungfudeathgrip); Release();
Shutdown(); Shutdown();
} }

Просмотреть файл

@ -91,7 +91,7 @@ GMPVideoEncoderParent::Close()
// In case this is the last reference // In case this is the last reference
nsRefPtr<GMPVideoEncoderParent> kungfudeathgrip(this); nsRefPtr<GMPVideoEncoderParent> kungfudeathgrip(this);
NS_RELEASE(kungfudeathgrip); Release();
Shutdown(); Shutdown();
} }