[M145] Revert "Fix Minimize operation on Wayland" [chromium/src : refs/branch-heads/7632]

0 views
Skip to first unread message

Rubber Stamper (Gerrit)

unread,
Jan 20, 2026, 1:16:14 PM (13 days ago) Jan 20
to Chrome Cherry Picker, Patryk Chodur, AyeAye, max+watc...@igalia.com, nickdiego+wa...@igalia.com, ozone-...@chromium.org

Rubber Stamper voted

Bot-Commit+1
Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedLint
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: refs/branch-heads/7632
Gerrit-Change-Id: Iffb3dc473eb5366c6bf015ddc5c89955d0c41bf3
Gerrit-Change-Number: 7498791
Gerrit-PatchSet: 2
Gerrit-Reviewer: Chrome Cherry Picker <chrome-che...@chops-service-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Gerrit-CC: Patryk Chodur <pch...@google.com>
Gerrit-Comment-Date: Tue, 20 Jan 2026 18:16:06 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
Jan 20, 2026, 2:19:57 PM (12 days ago) Jan 20
to Chrome Cherry Picker, Patryk Chodur, Rubber Stamper, AyeAye, max+watc...@igalia.com, nickdiego+wa...@igalia.com, ozone-...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:
[M145] Revert "Fix Minimize operation on Wayland"

Original change's description:
> Revert "Fix Minimize operation on Wayland"
>
> This reverts commit 4a1346c532dc2a6491cbb0c2ea9a56aa561e4fa3.
>
> Reason for revert: The commit caused a regression. See https://crbug.com/476344114
>
> Original change's description:
> > Fix Minimize operation on Wayland
> >
> > On Wayland there's a race condition in which if a window is Minimized
> > and the configure event is triggered on a window but minimizing didn't
> > yet take place, the window changes state to normal and cancels the
> > minimize event.
> >
> > Bug: 458599317
> > Change-Id: I721bbb5c21892bf245abe37c3bc937f7ac3a0189
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7265065
> > Commit-Queue: Patryk Chodur <pch...@google.com>
> > Reviewed-by: Orko Garai <or...@igalia.com>
> > Cr-Commit-Position: refs/heads/main@{#1566951}
>
> Bug: 476344114
> Change-Id: Iffb3dc473eb5366c6bf015ddc5c89955d0c41bf3
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7485633
> Bot-Commit: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
> Reviewed-by: Kramer Ge <fang...@chromium.org>
> Commit-Queue: Kramer Ge <fang...@chromium.org>
> Auto-Submit: Patryk Chodur <pch...@google.com>
> Cr-Commit-Position: refs/heads/main@{#1571253}

(cherry picked from commit 9245f50cec8ac07452b9837e2992adea91a48811)
Bug: 477054704,476344114
Change-Id: Iffb3dc473eb5366c6bf015ddc5c89955d0c41bf3
Commit-Queue: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/7632@{#565}
Cr-Branched-From: 0bbdf2913883391365383b0a5dfe7bf9fd1a5213-refs/heads/main@{#1568190}
Files:
  • M ui/ozone/platform/wayland/host/wayland_toplevel_window.cc
Change size: S
Delta: 1 file changed, 5 insertions(+), 12 deletions(-)
Branch: refs/branch-heads/7632
Submit Requirements:
  • requirement satisfiedCode-Review: Bot-Commit+1 by Rubber Stamper
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: chromium/src
Gerrit-Branch: refs/branch-heads/7632
Gerrit-Change-Id: Iffb3dc473eb5366c6bf015ddc5c89955d0c41bf3
Gerrit-Change-Number: 7498791
Gerrit-PatchSet: 3
Gerrit-Reviewer: Chrome Cherry Picker <chrome-che...@chops-service-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages