Index: content/common/gpu/media/vt_video_decode_accelerator.h |
diff --git a/content/common/gpu/media/vt_video_decode_accelerator.h b/content/common/gpu/media/vt_video_decode_accelerator.h |
index 327ab590e2b74d134d94ac61862a5596dea2c49d..efd76df5b6a5c402e9bfeefb1dd553a9415a2ffe 100644 |
--- a/content/common/gpu/media/vt_video_decode_accelerator.h |
+++ b/content/common/gpu/media/vt_video_decode_accelerator.h |
@@ -173,9 +173,9 @@ class VTVideoDecodeAccelerator : public media::VideoDecodeAccelerator { |
// Size of assigned picture buffers. |
gfx::Size picture_size_; |
- // Queue of frames that have not yet been decoded; maintains ownership of the |
- // Frame objects while they flow through VideoToolbox. |
- std::queue<linked_ptr<Frame>> pending_frames_; |
+ // Frames that have not yet been decoded; maintains ownership of Frame objects |
+ // while they flow through VideoToolbox. |
+ std::map<Frame*, linked_ptr<Frame>> pending_frames_; |
DaleCurtis
2014/11/22 00:10:51
Why not just an std::set?
sandersd (OOO until July 31)
2014/11/22 00:20:40
The linked_ptr is keeping the object alive in this
DaleCurtis
2014/11/22 00:25:06
This is really nasty though. Are you really saving
sandersd (OOO until July 31)
2014/11/22 00:32:48
No, but inside the task I'd have to have this same
DaleCurtis
2014/11/22 00:53:40
I don't follow why you'd change the tasks... they
sandersd (OOO until July 31)
2014/11/22 00:55:50
Sorry, that's DecodeTask().
DaleCurtis
2014/11/25 21:56:20
I still don't follow your arguments against set<Fr
|
// Set of assigned bitstream IDs, so that Destroy() can release them all. |
std::set<int32_t> assigned_bitstream_ids_; |