0

Use effective policies to determine if the session is curtained

In https://chromium-review.googlesource.com/c/chromium/src/+/5856503
we stopped applying setting the enable_curtaining flag on the
desktop environment options based on the local policy and moved to
using either session policies or reading the value from the
local policy provider.

In one instance, where we decide whether to apply a scale factor,
we were still reading the value from the desktop environment
options which was not correct and we use the default DPI.

Bug: 379171277
Change-Id: I8d2b83583578bfaf4cd0dca58d76cc4142274831
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6022065
Reviewed-by: Yuwei Huang <yuweih@chromium.org>
Auto-Submit: Joe Downing <joedow@chromium.org>
Commit-Queue: Yuwei Huang <yuweih@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1383334}
This commit is contained in:
Joe Downing
2024-11-15 00:04:45 +00:00
committed by Chromium LUCI CQ
parent 81c3b0370d
commit bd92210fd2

@ -158,7 +158,7 @@ void ClientSession::NotifyClientResolution(
webrtc::DesktopVector dpi_vector{kDefaultDpi, kDefaultDpi};
#if BUILDFLAG(IS_WIN)
// Matching the client DPI is only supported on Windows when curtained.
if (desktop_environment_options_.enable_curtaining()) {
if (effective_policies_.curtain_required.value_or(false)) {
dpi_vector.set(resolution.x_dpi(), resolution.y_dpi());
}
#elif BUILDFLAG(IS_LINUX)