Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1342)

Unified Diff: media/gpu/vt_video_decode_accelerator_mac.cc

Issue 2449993005: Remove GLImage::Destroy(). (Closed)
Patch Set: One more Destroy() call on Mac. Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/gpu/vaapi_tfp_picture.cc ('k') | ui/accelerated_widget_mac/ca_layer_tree_unittest_mac.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/gpu/vt_video_decode_accelerator_mac.cc
diff --git a/media/gpu/vt_video_decode_accelerator_mac.cc b/media/gpu/vt_video_decode_accelerator_mac.cc
index ce7ee9def408241249623c70fd57848c4f0a84f9..49a0a4fcff4c22e7b0ef975ed953b0d466b76ca1 100644
--- a/media/gpu/vt_video_decode_accelerator_mac.cc
+++ b/media/gpu/vt_video_decode_accelerator_mac.cc
@@ -283,10 +283,7 @@ VTVideoDecodeAccelerator::PictureInfo::PictureInfo(uint32_t client_texture_id,
: client_texture_id(client_texture_id),
service_texture_id(service_texture_id) {}
-VTVideoDecodeAccelerator::PictureInfo::~PictureInfo() {
- if (gl_image)
- gl_image->Destroy(false);
-}
+VTVideoDecodeAccelerator::PictureInfo::~PictureInfo() {}
bool VTVideoDecodeAccelerator::FrameOrder::operator()(
const linked_ptr<Frame>& lhs,
@@ -906,7 +903,6 @@ void VTVideoDecodeAccelerator::ReusePictureBuffer(int32_t picture_id) {
if (it != picture_info_map_.end()) {
PictureInfo* picture_info = it->second.get();
picture_info->cv_image.reset();
- picture_info->gl_image->Destroy(false);
picture_info->gl_image = nullptr;
}
« no previous file with comments | « media/gpu/vaapi_tfp_picture.cc ('k') | ui/accelerated_widget_mac/ca_layer_tree_unittest_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698