Chromium Code Reviews| Index: media/base/decoder_buffer.h |
| diff --git a/media/base/decoder_buffer.h b/media/base/decoder_buffer.h |
| index 665313ddcc1a698a23452c706355c80ed99f98d3..44eb4f9d46f0897cc6aa9b5dd0f32de4a89db1fd 100644 |
| --- a/media/base/decoder_buffer.h |
| +++ b/media/base/decoder_buffer.h |
| @@ -96,6 +96,7 @@ class MEDIA_EXPORT DecoderBuffer |
| return data_.get(); |
| } |
| + // TODO: data_size should return size_t instead of int |
|
ddorwin
2015/07/22 22:48:08
TODO's require a name:
TODO(servolk):...
servolk
2015/07/22 23:04:54
Done.
|
| int data_size() const { |
| DCHECK(!end_of_stream()); |
| return size_; |
| @@ -106,6 +107,7 @@ class MEDIA_EXPORT DecoderBuffer |
| return side_data_.get(); |
| } |
| + // TODO: side_data_size should return size_t instead of int |
| int side_data_size() const { |
| DCHECK(!end_of_stream()); |
| return side_data_size_; |
| @@ -187,6 +189,7 @@ class MEDIA_EXPORT DecoderBuffer |
| base::TimeDelta timestamp_; |
| base::TimeDelta duration_; |
| + // TODO: We should consider changing size_/side_data_size_ types to size_t. |
| int size_; |
| scoped_ptr<uint8, base::AlignedFreeDeleter> data_; |
| int side_data_size_; |