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

Issue 2111043002: Enable NV12 DXGI video copying path by default. (Closed)

Created:
4 years, 5 months ago by jbauman
Modified:
4 years, 5 months ago
Reviewers:
piman
CC:
chromium-reviews, darin-cc_chromium.org, jam
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Enable NV12 DXGI video copying path by default. Enables using NV12 textures as video decoder picture buffers on systems where that's supported - those that use D3D11 video decoding, support NV12 texture sharing, and can use NV12 as video processor outputs. The additional copy uses extra power over the new zero-copy DXGI path, but should work on systems where that hits driver issues, either due to sharing of the decoder device with ANGLE or with using decoder output textures as shader resources. BUG=574292 Committed: https://crrev.com/71e41e59f0ab10ec4a2998ca9ced81d8c67fd114 Cr-Commit-Position: refs/heads/master@{#403334}

Patch Set 1 #

Patch Set 2 : fix flag #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M content/public/browser/gpu_utils.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/public/common/content_switches.h View 1 chunk +1 line, -1 line 0 comments Download
M content/public/common/content_switches.cc View 1 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 9 (4 generated)
jbauman
4 years, 5 months ago (2016-06-30 03:12:20 UTC) #2
piman
lgtm
4 years, 5 months ago (2016-06-30 03:39:17 UTC) #3
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/2111043002/20001
4 years, 5 months ago (2016-06-30 22:38:21 UTC) #6
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 5 months ago (2016-06-30 23:29:44 UTC) #7
commit-bot: I haz the power
4 years, 5 months ago (2016-06-30 23:32:52 UTC) #9
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/71e41e59f0ab10ec4a2998ca9ced81d8c67fd114
Cr-Commit-Position: refs/heads/master@{#403334}

Powered by Google App Engine
This is Rietveld 408576698