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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 16430003: Remove linux x86 from HW video decode black list

Created:
7 years, 6 months ago by haltonhuo
Modified:
5 years, 6 months ago
Reviewers:
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Remove linux x86 from HW video decode black list BUG=137247 TEST=ppapi_example_video_decode on linux

Patch Set 1 #

Total comments: 3

Patch Set 2 : Rebase to r250909 #

Patch Set 3 : Update to disable chromeos checking in IsAcceleratedVideoDecodeEnabled() in bpf_gpu_policy_linux.cc #

Patch Set 4 : rebase to r257620 #

Patch Set 5 : rebase to r263658 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -7 lines) Patch
M content/common/gpu/media/gpu_video_decode_accelerator.cc View 1 2 3 4 2 chunks +2 lines, -2 lines 0 comments Download
M content/common/gpu/media/video_decode_accelerator_unittest.cc View 1 2 3 4 2 chunks +2 lines, -2 lines 0 comments Download
M content/content_common.gypi View 1 2 3 4 1 chunk +1 line, -1 line 0 comments Download
M gpu/config/software_rendering_list_json.cc View 1 2 3 4 2 chunks +7 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
haltonhuo
Since there is no plan to enable vaapi HW video decode on linux(get details from ...
7 years, 6 months ago (2013-06-05 07:51:58 UTC) #1
Zhenyao Mo
https://codereview.chromium.org/16430003/diff/1/gpu/config/software_rendering_list_json.cc File gpu/config/software_rendering_list_json.cc (right): https://codereview.chromium.org/16430003/diff/1/gpu/config/software_rendering_list_json.cc#newcode91 gpu/config/software_rendering_list_json.cc:91: "version": "6.0", please update the version to 6.1 https://codereview.chromium.org/16430003/diff/1/gpu/config/software_rendering_list_json.cc#newcode718 ...
7 years, 5 months ago (2013-07-16 23:03:38 UTC) #2
Ami GONE FROM CHROMIUM
@zmo: FYI this patch is not for landing (see haltonhuo@'s first message in the reviewlog)
7 years, 5 months ago (2013-07-23 05:37:57 UTC) #3
Ami GONE FROM CHROMIUM
7 years, 1 month ago (2013-10-31 21:26:38 UTC) #4 
Removing myself from reviewer list to drop this from my rietveld console.

Powered by Google App Engine
This is Rietveld 408576698