[mutter] xwayland: Split Xwayland display name getter into two



commit d8e75027a4ab7939e8d21c95174dea2b2b9c813f
Author: Jonas Ã…dahl <jadahl gmail com>
Date:   Thu Jan 21 18:56:18 2021 +0100

    xwayland: Split Xwayland display name getter into two
    
    One for the public channel, and one for the private maintainance
    channel. Use the public one for test clients, otherwise tests become
    flaky, and the private one for MetaX11Display.
    
    Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1681>

 src/tests/test-utils.c     | 2 +-
 src/wayland/meta-wayland.c | 8 +++++++-
 src/wayland/meta-wayland.h | 4 +++-
 src/x11/meta-x11-display.c | 2 +-
 4 files changed, 12 insertions(+), 4 deletions(-)
---
diff --git a/src/tests/test-utils.c b/src/tests/test-utils.c
index 28cf97164e..dc8de3175c 100644
--- a/src/tests/test-utils.c
+++ b/src/tests/test-utils.c
@@ -440,7 +440,7 @@ test_client_new (const char          *id,
   g_assert (meta_is_wayland_compositor ());
   compositor = meta_wayland_compositor_get_default ();
   wayland_display_name = meta_wayland_get_wayland_display_name (compositor);
-  x11_display_name = meta_wayland_get_xwayland_display_name (compositor);
+  x11_display_name = meta_wayland_get_public_xwayland_display_name (compositor);
 
   g_subprocess_launcher_setenv (launcher,
                                 "WAYLAND_DISPLAY", wayland_display_name,
diff --git a/src/wayland/meta-wayland.c b/src/wayland/meta-wayland.c
index 7da43d911c..ef7a495b5c 100644
--- a/src/wayland/meta-wayland.c
+++ b/src/wayland/meta-wayland.c
@@ -503,7 +503,13 @@ meta_wayland_get_wayland_display_name (MetaWaylandCompositor *compositor)
 }
 
 const char *
-meta_wayland_get_xwayland_display_name (MetaWaylandCompositor *compositor)
+meta_wayland_get_public_xwayland_display_name (MetaWaylandCompositor *compositor)
+{
+  return compositor->xwayland_manager.public_connection.name;
+}
+
+const char *
+meta_wayland_get_private_xwayland_display_name (MetaWaylandCompositor *compositor)
 {
   return compositor->xwayland_manager.private_connection.name;
 }
diff --git a/src/wayland/meta-wayland.h b/src/wayland/meta-wayland.h
index e59bb97bc4..454330e4f9 100644
--- a/src/wayland/meta-wayland.h
+++ b/src/wayland/meta-wayland.h
@@ -70,7 +70,9 @@ META_EXPORT_TEST
 const char             *meta_wayland_get_wayland_display_name   (MetaWaylandCompositor *compositor);
 
 META_EXPORT_TEST
-const char             *meta_wayland_get_xwayland_display_name  (MetaWaylandCompositor *compositor);
+const char             *meta_wayland_get_public_xwayland_display_name  (MetaWaylandCompositor *compositor);
+
+const char             *meta_wayland_get_private_xwayland_display_name (MetaWaylandCompositor *compositor);
 
 void                    meta_wayland_compositor_restore_shortcuts      (MetaWaylandCompositor *compositor,
                                                                         ClutterInputDevice    *source);
diff --git a/src/x11/meta-x11-display.c b/src/x11/meta-x11-display.c
index ce62a6a173..f11684f14d 100644
--- a/src/x11/meta-x11-display.c
+++ b/src/x11/meta-x11-display.c
@@ -1044,7 +1044,7 @@ meta_x11_get_display_name (void)
 
       compositor = meta_wayland_compositor_get_default ();
 
-      return meta_wayland_get_xwayland_display_name (compositor);
+      return meta_wayland_get_private_xwayland_display_name (compositor);
     }
   else
 #endif


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]