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

Issue 842123002: Support multiple video decoders and encoders (Closed)

Created:
5 years, 11 months ago by henryhsu
Modified:
5 years, 11 months ago
CC:
chromium-reviews, posciak+watch_chromium.org, jam, mcasas+watch_chromium.org, feature-media-reviews_chromium.org, darin-cc_chromium.org, piman+watch_chromium.org, wjia+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Support multiple video decoders and encoders Some platforms have multiple video decoders and encoders. For decode and encoder, return the first succeed initialized VDA and VEA from all possible platforms. GetSupportedProfile return all possible profiles from all encoders. This cl is submitted again because crrev.com/826663002 was reverted in crrev.com/832383004. BUG=445016 TEST=Tested on squawks. For decoder, make sure V4L2 initialization failed and VAAPI successed. For encoder, test on extension. TBR=piman@chromium.org,posciak@chromium.org Committed: https://crrev.com/712f1acaf22f225a5eaef28b3dff9aecb7bc5cc8 Cr-Commit-Position: refs/heads/master@{#310754}

Patch Set 1 #

Patch Set 2 : Add USE_V4L2_CODEC define #

Unified diffs Side-by-side diffs Delta from patch set Stats (+234 lines, -94 lines) Patch
M content/common/BUILD.gn View 1 1 chunk +4 lines, -1 line 0 comments Download
M content/common/gpu/media/gpu_video_decode_accelerator.h View 1 chunk +17 lines, -2 lines 0 comments Download
M content/common/gpu/media/gpu_video_decode_accelerator.cc View 1 2 chunks +102 lines, -52 lines 0 comments Download
M content/common/gpu/media/gpu_video_encode_accelerator.h View 1 chunk +11 lines, -5 lines 0 comments Download
M content/common/gpu/media/gpu_video_encode_accelerator.cc View 1 4 chunks +70 lines, -33 lines 0 comments Download
M content/common/gpu/media/v4l2_image_processor.h View 3 chunks +3 lines, -0 lines 0 comments Download
M content/common/gpu/media/v4l2_image_processor.cc View 1 chunk +9 lines, -0 lines 0 comments Download
M content/common/gpu/media/v4l2_video_device.cc View 2 chunks +4 lines, -0 lines 0 comments Download
M content/common/gpu/media/v4l2_video_encode_accelerator.h View 2 chunks +2 lines, -0 lines 0 comments Download
M content/common/gpu/media/v4l2_video_encode_accelerator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M content/content_common.gypi View 1 1 chunk +6 lines, -1 line 0 comments Download

Messages

Total messages: 9 (2 generated)
henryhsu
5 years, 11 months ago (2015-01-09 07:00:12 UTC) #2
wuchengli
LGTM
5 years, 11 months ago (2015-01-09 07:06:59 UTC) #3
piman
5 years, 11 months ago (2015-01-09 07:19:33 UTC) #4
piman
lgtm
5 years, 11 months ago (2015-01-09 07:19:34 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/842123002/20001
5 years, 11 months ago (2015-01-09 09:51:34 UTC) #7
commit-bot: I haz the power
Committed patchset #2 (id:20001)
5 years, 11 months ago (2015-01-09 11:26:16 UTC) #8
commit-bot: I haz the power
5 years, 11 months ago (2015-01-09 11:27:09 UTC) #9
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/712f1acaf22f225a5eaef28b3dff9aecb7bc5cc8
Cr-Commit-Position: refs/heads/master@{#310754}

Powered by Google App Engine
This is Rietveld 408576698