diff options
| author | dmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-30 16:48:43 +0000 |
|---|---|---|
| committer | dmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-30 16:48:43 +0000 |
| commit | 7835114c9a13f893a6c7c2452d3fc5b8f8768aa4 (patch) | |
| tree | 909bdf332289451b2ab7a4d7f2eaa568a3c36d7b /content/renderer/pepper/ppb_video_decoder_impl.cc | |
| parent | 700e8f48f51550e6a8b42892fc88153bd24de1c3 (diff) | |
| download | chromium_src-7835114c9a13f893a6c7c2452d3fc5b8f8768aa4.zip chromium_src-7835114c9a13f893a6c7c2452d3fc5b8f8768aa4.tar.gz chromium_src-7835114c9a13f893a6c7c2452d3fc5b8f8768aa4.tar.bz2 | |
Make DefaultDeleter for Video{De|En}codeAccelerator
This makes a specialization of base::DefaultDeleter so that scoped_ptr will
use Destroy() instead of the destructor. Without this, it seems wrong to
hold VideoDecodeAccelerator in a scoped_ptr, given that letting scoped_ptr
delete it via the destructor would be an error.
Also moves the destructors to protected, to prevent mistakes.
BUG=
Review URL: https://codereview.chromium.org/292183011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273860 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/renderer/pepper/ppb_video_decoder_impl.cc')
| -rw-r--r-- | content/renderer/pepper/ppb_video_decoder_impl.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/content/renderer/pepper/ppb_video_decoder_impl.cc b/content/renderer/pepper/ppb_video_decoder_impl.cc index 9844c36..4a3cde4 100644 --- a/content/renderer/pepper/ppb_video_decoder_impl.cc +++ b/content/renderer/pepper/ppb_video_decoder_impl.cc @@ -224,8 +224,7 @@ int32_t PPB_VideoDecoder_Impl::Reset(scoped_refptr<TrackedCallback> callback) { void PPB_VideoDecoder_Impl::Destroy() { FlushCommandBuffer(); - if (decoder_) - decoder_.release()->Destroy(); + decoder_.reset(); ppp_videodecoder_ = NULL; ::ppapi::PPB_VideoDecoder_Shared::Destroy(); |