commit 962b5fa3678a5d165c8c4230fab072315649140f
author Andrew Wolfers <aswolfers@chromium.org> Tue Aug 27 12:46:25 2024
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Aug 27 12:46:25 2024
tree 381bd7eeb6373d6c847fe61ad29fb84bcf4b081a
parent 7a1cd3421ecd72e930de77ea9950b467fbced5cf

Fix unnecessary window restacking

This CL eliminates unneeded window restacking from being initiated by
the FocusController. Prior to this CL, as part of focusing a window,
the active window is always stacked at the top of its parent. After the
stacking order changes however, its further modified by moving all
transient children above the active window.

This CL adds logic to FocusController which checks whether the final
expected stacking order already exists (i.e. the active window is above
all non-transient children and below all transient children). If so,
then the restacking operation is skipped entirely.

This change resolves a bug associated with the experimental feature
CrosContentAdjustedRefreshRate. Prior to this change, display
configuration changes would cause the FocusController to trigger the
stack reorder, which would eventually reset the preferred refresh rate
and cause another configuration. This change breaks that cycle to fix
the issue.


Bug: b:361330085
Change-Id: If49aa95bcc896aac6a0953fc2c175775cf6128a0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5812173
Commit-Queue: Andrew Wolfers <aswolfers@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1347311}