Chromium Code Reviews| Index: content/common/gpu/media/android_video_encode_accelerator.cc |
| diff --git a/content/common/gpu/media/android_video_encode_accelerator.cc b/content/common/gpu/media/android_video_encode_accelerator.cc |
| index d84673d6371e0c8a190a2ffc5b6cf4d3e415b4f7..4c1eaaa6c0c77430c9064e3596bcf9cc2474ea69 100644 |
| --- a/content/common/gpu/media/android_video_encode_accelerator.cc |
| +++ b/content/common/gpu/media/android_video_encode_accelerator.cc |
| @@ -335,7 +335,9 @@ void AndroidVideoEncodeAccelerator::QueueInput() { |
| uint8_t* buffer = NULL; |
| size_t capacity = 0; |
| - media_codec_->GetInputBuffer(input_buf_index, &buffer, &capacity); |
| + status = media_codec_->GetInputBuffer(input_buf_index, &buffer, &capacity); |
| + RETURN_ON_FAILURE(status != media::MEDIA_CODEC_ERROR, |
|
DaleCurtis
2016/03/04 20:33:34
Use == OK for readability.
|
| + "GetInputBuffer failed.", kPlatformFailureError); |
| size_t queued_size = |
| VideoFrame::AllocationSize(media::PIXEL_FORMAT_I420, frame->coded_size()); |
| @@ -429,7 +431,10 @@ void AndroidVideoEncodeAccelerator::DequeueOutput() { |
| << shm->mapped_size(), |
| kPlatformFailureError); |
| - media_codec_->CopyFromOutputBuffer(buf_index, offset, shm->memory(), size); |
| + media::MediaCodecStatus status = media_codec_->CopyFromOutputBuffer( |
| + buf_index, offset, shm->memory(), size); |
| + RETURN_ON_FAILURE(status == media::MEDIA_CODEC_OK, |
| + "CopyFromOutputBuffer failed", kPlatformFailureError); |
| media_codec_->ReleaseOutputBuffer(buf_index, false); |
| --num_buffers_at_codec_; |