Commit cd7a0f44 authored by Maggie Chen's avatar Maggie Chen Committed by Chromium LUCI CQ

Correct DirectCompositionSurfaceWin::AllowTearing() function

The Boolean result of this function was accidentally reversed in
https://chromium-review.googlesource.com/c/chromium/src/+/2597637.
This CL corrects the logic.

For some reason, this wrong AllowTearing flag prevents hardware overlay
promotions on Surface Book while the other laptop like Blazer is fine.
These two are the only devices I checked.

Bug: 43611
Change-Id: Ib6f27d74d55e0977df840e1f10001ccc01ca5227
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2606292Reviewed-by: default avatarZhenyao Mo <zmo@chromium.org>
Commit-Queue: Maggie Chen <magchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#839804}
parent f1d85da8
......@@ -653,7 +653,7 @@ bool DirectCompositionSurfaceWin::IsSwapChainTearingSupported() {
// static
bool DirectCompositionSurfaceWin::AllowTearing() {
// Swap chain tearing is used only if vsync is disabled explicitly.
return features::UseGpuVsync() &&
return !features::UseGpuVsync() &&
DirectCompositionSurfaceWin::IsSwapChainTearingSupported();
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment