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

Issue 9724030: Description: Video decode in hardware on ARM platform. (Closed)

Created:
8 years, 9 months ago by Arun M
Modified:
8 years, 9 months ago
CC:
chromium-reviews, joi+watch-content_chromium.org, feature-media-reviews_chromium.org, jam, apatrick_chromium, darin-cc_chromium.org
Visibility:
Public.

Description

Description: Video decode in hardware on ARM platform. This patch modifies the GpuVideoDecodeAccelerator machinery to enable HW decode on certain platforms. In case of some ARM platforms, the texture memory allocated by GPU(glTexImage2D) cannot be shared with other devices such as a seperate HW decoder. So a Pixmap buffer is allocated and an EGL Image is created from the pixmap handle. This EGL Image is sharable between the HW Decoder and the GPU. TEST=omx_video_decode_accelerator_unittest Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=127554

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+111 lines, -41 lines) Patch
M AUTHORS View 1 chunk +1 line, -0 lines 0 comments Download
M content/common/gpu/media/gles2_texture_to_egl_image_translator.h View 2 chunks +8 lines, -6 lines 0 comments Download
M content/common/gpu/media/gles2_texture_to_egl_image_translator.cc View 4 chunks +56 lines, -11 lines 0 comments Download
M content/common/gpu/media/omx_video_decode_accelerator.h View 4 chunks +8 lines, -1 line 0 comments Download
M content/common/gpu/media/omx_video_decode_accelerator.cc View 9 chunks +37 lines, -22 lines 0 comments Download
M content/common/gpu/media/video_decode_accelerator_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
Arun M
Please refer to https://chromiumcodereview.appspot.com/9346012/
8 years, 9 months ago (2012-03-19 04:38:14 UTC) #1
Ami GONE FROM CHROMIUM
lgtm
8 years, 9 months ago (2012-03-19 17:13:01 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/arun.m@samsung.com/9724030/1
8 years, 9 months ago (2012-03-19 17:13:13 UTC) #3
commit-bot: I haz the power
Try job failure for 9724030-1 (retry) on linux_rel for step "browser_tests". It's a second try, ...
8 years, 9 months ago (2012-03-19 18:22:08 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/arun.m@samsung.com/9724030/1
8 years, 9 months ago (2012-03-19 18:27:27 UTC) #5
commit-bot: I haz the power
Try job failure for 9724030-1 (retry) on mac_rel for step "browser_tests". It's a second try, ...
8 years, 9 months ago (2012-03-19 20:12:05 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/arun.m@samsung.com/9724030/1
8 years, 9 months ago (2012-03-19 20:13:50 UTC) #7
commit-bot: I haz the power
8 years, 9 months ago (2012-03-19 22:15:55 UTC) #8
Change committed as 127554

Powered by Google App Engine
This is Rietveld 408576698