File _service:obs_scm:revert-mutter-wayland-regressi... of Package ungoogled-chromium

Overview Repositories Revisions Requests Users Attributes Meta

File _service:obs_scm:revert-mutter-wayland-regression.patch of Package ungoogled-chromium (Revision 2405c630765c6a656ebb9681c1e378fb)

Currently displaying revision 2405c630765c6a656ebb9681c1e378fb , Show latest

From ac7d143e1d9eca969b3f23a7fa9d022577ae6005 Mon Sep 17 00:00:00 2001
From: Alexander Dunaev <adunaev@igalia.com>
Date: Mon, 15 Aug 2022 11:12:21 +0000
Subject: [PATCH] Revert "wayland: Commit surface after configure with same size"

This reverts commit 32e27b753bfb2e99232c1bb303f07283ab248ea3.

Reason for revert: this change caused a major regression on mutter Wayland compositor.

Original change's description:
> wayland: Commit surface after configure with same size
>
> The configure -> ack sequence doesn't take effect on the server side
> until the associated surface is committed.
>
> Fixes a sync issue in TabDragging/* tests on lacros that reposition
> windows during test setup, where lacros and exo have different screen
> positions for a surface.
>
> Bug: 1336691, 1336706
> Change-Id: I8d34f23ece4808bb56d8bc01712d4488bf472359
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3714566
> Reviewed-by: Kramer Ge <fangzhoug@chromium.org>
> Commit-Queue: River Gilhuly <rivr@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1023916}

Bug: 1336691, 1336706
Change-Id: Ia88538e5645da5a65d0c1073c672f1f3bfbc4c54
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3831855
Auto-Submit: Alexander Dunaev <adunaev@igalia.com>
Commit-Queue: Maksim Sisov <msisov@igalia.com>
Reviewed-by: Maksim Sisov <msisov@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1035020}
---

diff --git a/ui/ozone/platform/wayland/host/wayland_window.cc b/ui/ozone/platform/wayland/host/wayland_window.cc
index f4e616b2..fb95e498 100644
--- a/ui/ozone/platform/wayland/host/wayland_window.cc
+++ b/ui/ozone/platform/wayland/host/wayland_window.cc
@@ -950,7 +950,7 @@
     // window has been applied.
     SetWindowGeometry(pending_bounds_dip_);
     AckConfigure(serial);
-    root_surface()->Commit();
+    connection()->ScheduleFlush();
   } else if (!pending_configures_.empty() &&
              pending_bounds_dip_.size() ==
                  pending_configures_.back().bounds_dip.size()) {
diff --git a/ui/ozone/platform/wayland/wayland_buffer_manager_unittest.cc b/ui/ozone/platform/wayland/wayland_buffer_manager_unittest.cc
index 7a1a95d..58804a0 100644
--- a/ui/ozone/platform/wayland/wayland_buffer_manager_unittest.cc
+++ b/ui/ozone/platform/wayland/wayland_buffer_manager_unittest.cc
@@ -1112,9 +1112,7 @@
     EXPECT_CALL(*xdg_surface, AckConfigure(_)).Times(1);
     EXPECT_CALL(*mock_surface, Attach(_, _, _)).Times(1);
     EXPECT_CALL(*mock_surface, Frame(_)).Times(1);
-    // Commit() can be called a second time as part of the configure -> ack
-    // sequence.
-    EXPECT_CALL(*mock_surface, Commit()).Times(testing::Between(1, 2));
+    EXPECT_CALL(*mock_surface, Commit()).Times(1);
 
     ActivateSurface(mock_surface->xdg_surface());
     Sync();

Places

File _service:obs_scm:revert-mutter-wayland-regression.patch of Package ungoogled-chromium (Revision 2405c630765c6a656ebb9681c1e378fb)

Places