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

Unified Diff: content/common/gpu/media/omx_video_decode_accelerator.cc

Issue 8897022: Revert 113895 - <video> decode in hardware! (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years 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
Index: content/common/gpu/media/omx_video_decode_accelerator.cc
===================================================================
--- content/common/gpu/media/omx_video_decode_accelerator.cc (revision 113906)
+++ content/common/gpu/media/omx_video_decode_accelerator.cc (working copy)
@@ -50,25 +50,25 @@
// Maps h264-related Profile enum values to OMX_VIDEO_AVCPROFILETYPE values.
static OMX_U32 MapH264ProfileToOMXAVCProfile(uint32 profile) {
switch (profile) {
- case media::H264PROFILE_BASELINE:
+ case media::VideoDecodeAccelerator::H264PROFILE_BASELINE:
return OMX_VIDEO_AVCProfileBaseline;
- case media::H264PROFILE_MAIN:
+ case media::VideoDecodeAccelerator::H264PROFILE_MAIN:
return OMX_VIDEO_AVCProfileMain;
- case media::H264PROFILE_EXTENDED:
+ case media::VideoDecodeAccelerator::H264PROFILE_EXTENDED:
return OMX_VIDEO_AVCProfileExtended;
- case media::H264PROFILE_HIGH:
+ case media::VideoDecodeAccelerator::H264PROFILE_HIGH:
return OMX_VIDEO_AVCProfileHigh;
- case media::H264PROFILE_HIGH10PROFILE:
+ case media::VideoDecodeAccelerator::H264PROFILE_HIGH10PROFILE:
return OMX_VIDEO_AVCProfileHigh10;
- case media::H264PROFILE_HIGH422PROFILE:
+ case media::VideoDecodeAccelerator::H264PROFILE_HIGH422PROFILE:
return OMX_VIDEO_AVCProfileHigh422;
- case media::H264PROFILE_HIGH444PREDICTIVEPROFILE:
+ case media::VideoDecodeAccelerator::H264PROFILE_HIGH444PREDICTIVEPROFILE:
return OMX_VIDEO_AVCProfileHigh444;
// Below enums don't have equivalent enum in Openmax.
- case media::H264PROFILE_SCALABLEBASELINE:
- case media::H264PROFILE_SCALABLEHIGH:
- case media::H264PROFILE_STEREOHIGH:
- case media::H264PROFILE_MULTIVIEWHIGH:
+ case media::VideoDecodeAccelerator::H264PROFILE_SCALABLEBASELINE:
+ case media::VideoDecodeAccelerator::H264PROFILE_SCALABLEHIGH:
+ case media::VideoDecodeAccelerator::H264PROFILE_STEREOHIGH:
+ case media::VideoDecodeAccelerator::H264PROFILE_MULTIVIEWHIGH:
// Nvidia OMX video decoder requires the same resources (as that of the
// High profile) in every profile higher to the Main profile.
return OMX_VIDEO_AVCProfileHigh444;
@@ -144,8 +144,7 @@
bool OmxVideoDecodeAccelerator::Initialize(Profile profile) {
DCHECK_EQ(message_loop_, MessageLoop::current());
- RETURN_ON_FAILURE((profile >= media::H264PROFILE_MIN &&
- profile <= media::H264PROFILE_MAX),
+ RETURN_ON_FAILURE(profile >= H264PROFILE_MIN && profile <= H264PROFILE_MAX,
"Only h264 supported", INVALID_ARGUMENT, false);
profile_ = MapH264ProfileToOMXAVCProfile(profile);
RETURN_ON_FAILURE(profile_ != OMX_VIDEO_AVCProfileMax,
@@ -769,8 +768,7 @@
--output_buffers_at_component_;
if (fake_output_buffers_.size() && fake_output_buffers_.count(buffer)) {
- size_t erased = fake_output_buffers_.erase(buffer);
- DCHECK_EQ(erased, 1U);
+ DCHECK_EQ(fake_output_buffers_.erase(buffer), 1U);
OMX_ERRORTYPE result =
OMX_FreeBuffer(component_handle_, output_port_, buffer);
RETURN_ON_OMX_FAILURE(result, "OMX_FreeBuffer failed", PLATFORM_FAILURE,);
Property changes on: content/common/gpu/media/omx_video_decode_accelerator.cc
___________________________________________________________________
Added: svn:mergeinfo

Powered by Google App Engine
This is Rietveld 408576698