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

Issue 2902313003: Disable hardware acceleration for vp9 on Win 10 until Creators Update. (Closed)

Created:
3 years, 7 months ago by Igor Vayzert
Modified:
3 years, 6 months ago
Reviewers:
DaleCurtis, *jbauman
CC:
chromium-reviews, feature-media-reviews_chromium.org, piman+watch_chromium.org, posciak+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Disable hardware acceleration for vp9 on Win 10 until Creators Update. R=dalecurtis@chromium.org BUG=667532 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2902313003 Cr-Commit-Position: refs/heads/master@{#475769} Committed: https://chromium.googlesource.com/chromium/src/+/e79d64564bb28322975a1b7b812d2c9331bd14ee

Patch Set 1 #

Patch Set 2 : Use blacklist file. #

Patch Set 3 : Use blacklist file. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M gpu/config/gpu_driver_bug_list.json View 1 2 2 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 13 (6 generated)
DaleCurtis
Do we only want to do this with nvidia drivers? +jbauman
3 years, 7 months ago (2017-05-25 18:25:04 UTC) #4
jbauman
On 2017/05/25 18:25:04, DaleCurtis wrote: > Do we only want to do this with nvidia ...
3 years, 7 months ago (2017-05-25 19:21:39 UTC) #5
Igor Vayzert
On 2017/05/25 19:21:39, jbauman wrote: > On 2017/05/25 18:25:04, DaleCurtis wrote: > > Do we ...
3 years, 7 months ago (2017-05-26 07:07:52 UTC) #6
DaleCurtis
Nah, there's very little vp8 content that needs hardware decode, so this is fine.
3 years, 7 months ago (2017-05-26 16:40:17 UTC) #7
jbauman
lgtm
3 years, 6 months ago (2017-05-30 23:10:28 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2902313003/40001
3 years, 6 months ago (2017-05-31 01:48:36 UTC) #10
commit-bot: I haz the power
3 years, 6 months ago (2017-05-31 03:29:59 UTC) #13
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/e79d64564bb28322975a1b7b812d...

Powered by Google App Engine
This is Rietveld 408576698