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

Side by Side Diff: media/filters/gpu_video_decoder.h

Issue 313623003: WIP: Option2: Android media: VideoFrame should not store so many sync points. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 6 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 unified diff | Download patch
« no previous file with comments | « media/base/video_frame_unittest.cc ('k') | media/filters/gpu_video_decoder.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef MEDIA_FILTERS_GPU_VIDEO_DECODER_H_ 5 #ifndef MEDIA_FILTERS_GPU_VIDEO_DECODER_H_
6 #define MEDIA_FILTERS_GPU_VIDEO_DECODER_H_ 6 #define MEDIA_FILTERS_GPU_VIDEO_DECODER_H_
7 7
8 #include <list> 8 #include <list>
9 #include <map> 9 #include <map>
10 #include <set> 10 #include <set>
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 // empty. 102 // empty.
103 void EnqueueFrameAndTriggerFrameDelivery( 103 void EnqueueFrameAndTriggerFrameDelivery(
104 const scoped_refptr<VideoFrame>& frame); 104 const scoped_refptr<VideoFrame>& frame);
105 105
106 // Static method is to allow it to run even after GVD is deleted. 106 // Static method is to allow it to run even after GVD is deleted.
107 static void ReleaseMailbox( 107 static void ReleaseMailbox(
108 base::WeakPtr<GpuVideoDecoder> decoder, 108 base::WeakPtr<GpuVideoDecoder> decoder,
109 const scoped_refptr<media::GpuVideoAcceleratorFactories>& factories, 109 const scoped_refptr<media::GpuVideoAcceleratorFactories>& factories,
110 int64 picture_buffer_id, 110 int64 picture_buffer_id,
111 uint32 texture_id, 111 uint32 texture_id,
112 const std::vector<uint32>& release_sync_points); 112 const std::map<std::string, uint32>& release_sync_points);
113 // Indicate the picture buffer can be reused by the decoder. 113 // Indicate the picture buffer can be reused by the decoder.
114 void ReusePictureBuffer(int64 picture_buffer_id); 114 void ReusePictureBuffer(int64 picture_buffer_id);
115 115
116 void RecordBufferData( 116 void RecordBufferData(
117 const BitstreamBuffer& bitstream_buffer, const DecoderBuffer& buffer); 117 const BitstreamBuffer& bitstream_buffer, const DecoderBuffer& buffer);
118 void GetBufferData(int32 id, base::TimeDelta* timetamp, 118 void GetBufferData(int32 id, base::TimeDelta* timetamp,
119 gfx::Rect* visible_rect, gfx::Size* natural_size); 119 gfx::Rect* visible_rect, gfx::Size* natural_size);
120 120
121 void DestroyVDA(); 121 void DestroyVDA();
122 122
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
193 // Bound to factories_->GetMessageLoop(). 193 // Bound to factories_->GetMessageLoop().
194 // NOTE: Weak pointers must be invalidated before all other member variables. 194 // NOTE: Weak pointers must be invalidated before all other member variables.
195 base::WeakPtrFactory<GpuVideoDecoder> weak_factory_; 195 base::WeakPtrFactory<GpuVideoDecoder> weak_factory_;
196 196
197 DISALLOW_COPY_AND_ASSIGN(GpuVideoDecoder); 197 DISALLOW_COPY_AND_ASSIGN(GpuVideoDecoder);
198 }; 198 };
199 199
200 } // namespace media 200 } // namespace media
201 201
202 #endif // MEDIA_FILTERS_GPU_VIDEO_DECODER_H_ 202 #endif // MEDIA_FILTERS_GPU_VIDEO_DECODER_H_
OLDNEW
« no previous file with comments | « media/base/video_frame_unittest.cc ('k') | media/filters/gpu_video_decoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698