File mutter-window-actor-Special-case-shaped-Java-wi... of Package mutter

Overview Repositories Revisions Requests Users Attributes Meta

File mutter-window-actor-Special-case-shaped-Java-windows.patch of Package mutter (Revision 70c4a4206b089b852d92ca5b4223082f)

Currently displaying revision 70c4a4206b089b852d92ca5b4223082f , Show latest

From b3b5aa01c63aee1df079e0394b0e6372df1838d0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20M=C3=BCllner?= <fmuellner@gnome.org>
Date: Fri, 12 May 2017 13:40:31 +0200
Subject: [PATCH] window-actor: Special-case shaped Java windows

OpenJDK wrongly assumes that shaping a window implies no shadows.
They got lucky until commit b975676c changed the fallback case,
but now their compliance tests are broken. Make them happy again
by special-casing shaped Java windows.
---
 src/compositor/meta-window-actor-x11.c | 8 ++++++++
 src/x11/window-x11-private.h           | 2 ++
 src/x11/window-x11.c                   | 9 +++++++++
 3 files changed, 19 insertions(+)

Index: mutter-47.beta/src/compositor/meta-window-actor-x11.c
===================================================================
--- mutter-47.beta.orig/src/compositor/meta-window-actor-x11.c
+++ mutter-47.beta/src/compositor/meta-window-actor-x11.c
@@ -423,6 +423,14 @@ has_shadow (MetaWindowActorX11 *actor_x1
    */
   if (!meta_window_actor_is_opaque (META_WINDOW_ACTOR (actor_x11)))
     return FALSE;
+ 
+  /*
+   * OpenJDK wrongly assumes that shaping a window implies no compositor
+   * shadows; make its compliance tests happy to give it what it wants ...
+   */
+  if (g_strcmp0 (window->res_name, "sun-awt-X11-XWindowPeer") == 0 &&
+      meta_window_x11_is_shaped (window))
+    return FALSE;
 
   /*
    * If a window specifies that it has custom frame extents, that likely
Index: mutter-47.beta/src/x11/window-x11-private.h
===================================================================
--- mutter-47.beta.orig/src/x11/window-x11-private.h
+++ mutter-47.beta/src/x11/window-x11-private.h
@@ -128,6 +128,8 @@ gboolean meta_window_x11_has_pointer (Me
 gboolean meta_window_x11_same_application (MetaWindow *window,
                                            MetaWindow *other_window);
 
+gboolean meta_window_x11_is_shaped (MetaWindow *window);
+
 void meta_window_x11_shutdown_group (MetaWindow *window);
 
 META_EXPORT
Index: mutter-47.beta/src/x11/window-x11.c
===================================================================
--- mutter-47.beta.orig/src/x11/window-x11.c
+++ mutter-47.beta/src/x11/window-x11.c
@@ -2605,6 +2605,15 @@ meta_window_x11_update_shape_region (Met
   meta_window_set_shape_region (window, region);
 }
 
+gboolean
+meta_window_x11_is_shaped (MetaWindow *window)
+{
+  MetaWindowX11 *window_x11 = META_WINDOW_X11 (window);
+  MetaWindowX11Private *priv = meta_window_x11_get_instance_private (window_x11);
+
+  return priv->shape_region != NULL;
+}
+
 /* Generally meta_window_x11_same_application() is a better idea
  * of "sameness", since it handles the case where multiple apps
  * want to look like the same app or the same app wants to look

Places

File mutter-window-actor-Special-case-shaped-Java-windows.patch of Package mutter (Revision 70c4a4206b089b852d92ca5b4223082f)

Places