[mutter] window-actor: Move shapes, shadows and unredirection to X11 sub types



commit f7315c9a364621d7ba4d3ab291bf613f25ef0ad0
Author: Jonas Ådahl <jadahl gmail com>
Date:   Sat Aug 17 10:00:46 2019 +0200

    window-actor: Move shapes, shadows and unredirection to X11 sub types
    
    Move out updating of various shapes (input, opaque, shape) indirectly
    from X11 to the corresponding X11 sub types of MetaWindowActor and
    MetaSurfaceActor.
    
    Also move fullscreen window unredirection code with it. We want to
    effectively do something similar for MetaCompositorServer, but it will
    work differently enough not to share too much logic.
    
    While it would have been nice to move things piece by piece, things were
    too intertwined to make it feasible.
    
    This has the side effect fixing accidentally and arbitrarily adding
    server side shadow to Wayland surfaces.
    
    Fixes: https://gitlab.gnome.org/GNOME/mutter/issues/727
    
    https://gitlab.gnome.org/GNOME/mutter/merge_requests/734

 src/compositor/compositor.c                  |   19 +-
 src/compositor/meta-compositor-x11.c         |   16 +-
 src/compositor/meta-shaped-texture-private.h |    1 +
 src/compositor/meta-shaped-texture.c         |    2 +-
 src/compositor/meta-surface-actor-wayland.c  |   23 +-
 src/compositor/meta-surface-actor-x11.c      |   47 +-
 src/compositor/meta-surface-actor-x11.h      |    6 +
 src/compositor/meta-surface-actor.c          |   57 +-
 src/compositor/meta-surface-actor.h          |   13 +-
 src/compositor/meta-window-actor-private.h   |   17 +-
 src/compositor/meta-window-actor-wayland.c   |   28 +
 src/compositor/meta-window-actor-x11.c       | 1005 +++++++++++++++++++++++++-
 src/compositor/meta-window-actor-x11.h       |   13 +
 src/compositor/meta-window-actor.c           |  901 +----------------------
 14 files changed, 1111 insertions(+), 1037 deletions(-)
---
diff --git a/src/compositor/compositor.c b/src/compositor/compositor.c
index f7dcbd579..30a7a3824 100644
--- a/src/compositor/compositor.c
+++ b/src/compositor/compositor.c
@@ -677,7 +677,7 @@ meta_compositor_window_shape_changed (MetaCompositor *compositor,
   if (!window_actor)
     return;
 
-  meta_window_actor_update_shape (window_actor);
+  meta_window_actor_x11_update_shape (META_WINDOW_ACTOR_X11 (window_actor));
 }
 
 void
@@ -1149,18 +1149,6 @@ meta_post_paint_func (gpointer data)
   return TRUE;
 }
 
-static void
-on_shadow_factory_changed (MetaShadowFactory *factory,
-                           MetaCompositor    *compositor)
-{
-  MetaCompositorPrivate *priv =
-    meta_compositor_get_instance_private (compositor);
-  GList *l;
-
-  for (l = priv->windows; l; l = l->next)
-    meta_window_actor_invalidate_shadow (l->data);
-}
-
 static void
 meta_compositor_set_property (GObject      *object,
                               guint         prop_id,
@@ -1211,11 +1199,6 @@ meta_compositor_init (MetaCompositor *compositor)
 
   priv->context = clutter_backend->cogl_context;
 
-  g_signal_connect (meta_shadow_factory_get_default (),
-                    "changed",
-                    G_CALLBACK (on_shadow_factory_changed),
-                    compositor);
-
   priv->pre_paint_func_id =
     clutter_threads_add_repaint_func_full (CLUTTER_REPAINT_FLAGS_PRE_PAINT,
                                            meta_pre_paint_func,
diff --git a/src/compositor/meta-compositor-x11.c b/src/compositor/meta-compositor-x11.c
index b31ec451c..fce3f830c 100644
--- a/src/compositor/meta-compositor-x11.c
+++ b/src/compositor/meta-compositor-x11.c
@@ -29,6 +29,7 @@
 #include "backends/x11/meta-event-x11.h"
 #include "clutter/x11/clutter-x11.h"
 #include "compositor/meta-sync-ring.h"
+#include "compositor/meta-window-actor-x11.h"
 #include "core/display-private.h"
 #include "x11/meta-x11-display-private.h"
 
@@ -52,8 +53,9 @@ process_damage (MetaCompositorX11  *compositor_x11,
                 MetaWindow         *window)
 {
   MetaWindowActor *window_actor = meta_window_actor_from_window (window);
+  MetaWindowActorX11 *window_actor_x11 = META_WINDOW_ACTOR_X11 (window_actor);
 
-  meta_window_actor_process_x11_damage (window_actor, damage_xevent);
+  meta_window_actor_x11_process_damage (window_actor_x11, damage_xevent);
 
   compositor_x11->frame_has_updated_xsurfaces = TRUE;
 }
@@ -212,9 +214,11 @@ set_unredirected_window (MetaCompositorX11 *compositor_x11,
   if (prev_unredirected_window)
     {
       MetaWindowActor *window_actor;
+      MetaWindowActorX11 *window_actor_x11;
 
       window_actor = meta_window_actor_from_window (prev_unredirected_window);
-      meta_window_actor_set_unredirected (window_actor, FALSE);
+      window_actor_x11 = META_WINDOW_ACTOR_X11 (window_actor);
+      meta_window_actor_x11_set_unredirected (window_actor_x11, FALSE);
     }
 
   shape_cow_for_window (compositor_x11, window);
@@ -223,9 +227,11 @@ set_unredirected_window (MetaCompositorX11 *compositor_x11,
   if (window)
     {
       MetaWindowActor *window_actor;
+      MetaWindowActorX11 *window_actor_x11;
 
       window_actor = meta_window_actor_from_window (window);
-      meta_window_actor_set_unredirected (window_actor, TRUE);
+      window_actor_x11 = META_WINDOW_ACTOR_X11 (window_actor);
+      meta_window_actor_x11_set_unredirected (window_actor_x11, TRUE);
     }
 }
 
@@ -235,6 +241,7 @@ maybe_unredirect_top_window (MetaCompositorX11 *compositor_x11)
   MetaCompositor *compositor = META_COMPOSITOR (compositor_x11);
   MetaWindow *window_to_unredirect = NULL;
   MetaWindowActor *window_actor;
+  MetaWindowActorX11 *window_actor_x11;
 
   if (meta_compositor_is_unredirect_inhibited (compositor))
     goto out;
@@ -243,7 +250,8 @@ maybe_unredirect_top_window (MetaCompositorX11 *compositor_x11)
   if (!window_actor)
     goto out;
 
-  if (!meta_window_actor_should_unredirect (window_actor))
+  window_actor_x11 = META_WINDOW_ACTOR_X11 (window_actor);
+  if (!meta_window_actor_x11_should_unredirect (window_actor_x11))
     goto out;
 
   window_to_unredirect = meta_window_actor_get_meta_window (window_actor);
diff --git a/src/compositor/meta-shaped-texture-private.h b/src/compositor/meta-shaped-texture-private.h
index 58a102950..9b340777d 100644
--- a/src/compositor/meta-shaped-texture-private.h
+++ b/src/compositor/meta-shaped-texture-private.h
@@ -42,6 +42,7 @@ void meta_shaped_texture_set_fallback_size (MetaShapedTexture *stex,
                                             int                fallback_height);
 cairo_region_t * meta_shaped_texture_get_opaque_region (MetaShapedTexture *stex);
 gboolean meta_shaped_texture_is_opaque (MetaShapedTexture *stex);
+gboolean meta_shaped_texture_has_alpha (MetaShapedTexture *stex);
 void meta_shaped_texture_set_transform (MetaShapedTexture    *stex,
                                         MetaMonitorTransform  transform);
 void meta_shaped_texture_set_viewport_src_rect (MetaShapedTexture *stex,
diff --git a/src/compositor/meta-shaped-texture.c b/src/compositor/meta-shaped-texture.c
index 4cfe8af12..4b9e7864f 100644
--- a/src/compositor/meta-shaped-texture.c
+++ b/src/compositor/meta-shaped-texture.c
@@ -1019,7 +1019,7 @@ meta_shaped_texture_get_opaque_region (MetaShapedTexture *stex)
   return stex->opaque_region;
 }
 
-static gboolean
+gboolean
 meta_shaped_texture_has_alpha (MetaShapedTexture *stex)
 {
   CoglTexture *texture;
diff --git a/src/compositor/meta-surface-actor-wayland.c b/src/compositor/meta-surface-actor-wayland.c
index 1c933d554..0b1c4ef01 100644
--- a/src/compositor/meta-surface-actor-wayland.c
+++ b/src/compositor/meta-surface-actor-wayland.c
@@ -72,23 +72,11 @@ meta_surface_actor_wayland_is_visible (MetaSurfaceActor *actor)
 }
 
 static gboolean
-meta_surface_actor_wayland_should_unredirect (MetaSurfaceActor *actor)
+meta_surface_actor_wayland_is_opaque (MetaSurfaceActor *actor)
 {
-  return FALSE;
-}
-
-static void
-meta_surface_actor_wayland_set_unredirected (MetaSurfaceActor *actor,
-                                             gboolean          unredirected)
-{
-  /* Do nothing. In the future, we'll use KMS to set this
-   * up as a hardware overlay or something. */
-}
+  MetaShapedTexture *stex = meta_surface_actor_get_texture (actor);
 
-static gboolean
-meta_surface_actor_wayland_is_unredirected (MetaSurfaceActor *actor)
-{
-  return FALSE;
+  return meta_shaped_texture_is_opaque (stex);
 }
 
 void
@@ -162,10 +150,7 @@ meta_surface_actor_wayland_class_init (MetaSurfaceActorWaylandClass *klass)
   surface_actor_class->process_damage = meta_surface_actor_wayland_process_damage;
   surface_actor_class->pre_paint = meta_surface_actor_wayland_pre_paint;
   surface_actor_class->is_visible = meta_surface_actor_wayland_is_visible;
-
-  surface_actor_class->should_unredirect = meta_surface_actor_wayland_should_unredirect;
-  surface_actor_class->set_unredirected = meta_surface_actor_wayland_set_unredirected;
-  surface_actor_class->is_unredirected = meta_surface_actor_wayland_is_unredirected;
+  surface_actor_class->is_opaque = meta_surface_actor_wayland_is_opaque;
 
   surface_actor_class->get_window = meta_surface_actor_wayland_get_window;
 
diff --git a/src/compositor/meta-surface-actor-x11.c b/src/compositor/meta-surface-actor-x11.c
index a0275c6c3..64702f88a 100644
--- a/src/compositor/meta-surface-actor-x11.c
+++ b/src/compositor/meta-surface-actor-x11.c
@@ -249,33 +249,17 @@ static gboolean
 meta_surface_actor_x11_is_opaque (MetaSurfaceActor *actor)
 {
   MetaSurfaceActorX11 *self = META_SURFACE_ACTOR_X11 (actor);
+  MetaShapedTexture *stex = meta_surface_actor_get_texture (actor);
 
-  /* If we're not ARGB32, then we're opaque. */
-  if (!meta_surface_actor_is_argb32 (actor))
+  if (meta_surface_actor_x11_is_unredirected (self))
     return TRUE;
 
-  cairo_region_t *opaque_region = meta_surface_actor_get_opaque_region (actor);
-
-  /* If we have no opaque region, then no pixels are opaque. */
-  if (!opaque_region)
-    return FALSE;
-
-  MetaWindow *window = self->window;
-  cairo_rectangle_int_t client_area;
-  meta_window_get_client_area_rect (window, &client_area);
-
-  /* Otherwise, check if our opaque region covers our entire surface. */
-  if (cairo_region_contains_rectangle (opaque_region, &client_area) == CAIRO_REGION_OVERLAP_IN)
-    return TRUE;
-
-  return FALSE;
+  return meta_shaped_texture_is_opaque (stex);
 }
 
-static gboolean
-meta_surface_actor_x11_should_unredirect (MetaSurfaceActor *actor)
+gboolean
+meta_surface_actor_x11_should_unredirect (MetaSurfaceActorX11 *self)
 {
-  MetaSurfaceActorX11 *self = META_SURFACE_ACTOR_X11 (actor);
-
   MetaWindow *window = self->window;
 
   if (meta_window_requested_dont_bypass_compositor (window))
@@ -293,7 +277,7 @@ meta_surface_actor_x11_should_unredirect (MetaSurfaceActor *actor)
   if (meta_window_requested_bypass_compositor (window))
     return TRUE;
 
-  if (!meta_surface_actor_x11_is_opaque (actor))
+  if (!meta_surface_actor_x11_is_opaque (META_SURFACE_ACTOR (self)))
     return FALSE;
 
   if (meta_window_is_override_redirect (window))
@@ -327,12 +311,10 @@ sync_unredirected (MetaSurfaceActorX11 *self)
   meta_x11_error_trap_pop (display->x11_display);
 }
 
-static void
-meta_surface_actor_x11_set_unredirected (MetaSurfaceActor *actor,
-                                         gboolean          unredirected)
+void
+meta_surface_actor_x11_set_unredirected (MetaSurfaceActorX11 *self,
+                                         gboolean             unredirected)
 {
-  MetaSurfaceActorX11 *self = META_SURFACE_ACTOR_X11 (actor);
-
   if (self->unredirected == unredirected)
     return;
 
@@ -340,11 +322,9 @@ meta_surface_actor_x11_set_unredirected (MetaSurfaceActor *actor,
   sync_unredirected (self);
 }
 
-static gboolean
-meta_surface_actor_x11_is_unredirected (MetaSurfaceActor *actor)
+gboolean
+meta_surface_actor_x11_is_unredirected (MetaSurfaceActorX11 *self)
 {
-  MetaSurfaceActorX11 *self = META_SURFACE_ACTOR_X11 (actor);
-
   return self->unredirected;
 }
 
@@ -383,10 +363,7 @@ meta_surface_actor_x11_class_init (MetaSurfaceActorX11Class *klass)
   surface_actor_class->process_damage = meta_surface_actor_x11_process_damage;
   surface_actor_class->pre_paint = meta_surface_actor_x11_pre_paint;
   surface_actor_class->is_visible = meta_surface_actor_x11_is_visible;
-
-  surface_actor_class->should_unredirect = meta_surface_actor_x11_should_unredirect;
-  surface_actor_class->set_unredirected = meta_surface_actor_x11_set_unredirected;
-  surface_actor_class->is_unredirected = meta_surface_actor_x11_is_unredirected;
+  surface_actor_class->is_opaque = meta_surface_actor_x11_is_opaque;
 
   surface_actor_class->get_window = meta_surface_actor_x11_get_window;
 }
diff --git a/src/compositor/meta-surface-actor-x11.h b/src/compositor/meta-surface-actor-x11.h
index 2c4ed4dd6..4360065cf 100644
--- a/src/compositor/meta-surface-actor-x11.h
+++ b/src/compositor/meta-surface-actor-x11.h
@@ -46,6 +46,12 @@ MetaSurfaceActor * meta_surface_actor_x11_new (MetaWindow *window);
 
 void meta_surface_actor_x11_set_size (MetaSurfaceActorX11 *self,
                                       int width, int height);
+gboolean meta_surface_actor_x11_should_unredirect (MetaSurfaceActorX11 *self);
+
+void meta_surface_actor_x11_set_unredirected (MetaSurfaceActorX11 *self,
+                                              gboolean             unredirected);
+
+gboolean meta_surface_actor_x11_is_unredirected (MetaSurfaceActorX11 *self);
 
 G_END_DECLS
 
diff --git a/src/compositor/meta-surface-actor.c b/src/compositor/meta-surface-actor.c
index c910a145b..6fb3cf99d 100644
--- a/src/compositor/meta-surface-actor.c
+++ b/src/compositor/meta-surface-actor.c
@@ -221,6 +221,12 @@ meta_surface_actor_class_init (MetaSurfaceActorClass *klass)
                                         G_TYPE_NONE, 0);
 }
 
+gboolean
+meta_surface_actor_is_opaque (MetaSurfaceActor *self)
+{
+  return META_SURFACE_ACTOR_GET_CLASS (self)->is_opaque (self);
+}
+
 static void
 meta_surface_actor_cull_out (MetaCullable   *cullable,
                              cairo_region_t *unobscured_region,
@@ -481,38 +487,6 @@ meta_surface_actor_pre_paint (MetaSurfaceActor *self)
   META_SURFACE_ACTOR_GET_CLASS (self)->pre_paint (self);
 }
 
-gboolean
-meta_surface_actor_is_argb32 (MetaSurfaceActor *self)
-{
-  MetaShapedTexture *stex = meta_surface_actor_get_texture (self);
-  CoglTexture *texture = meta_shaped_texture_get_texture (stex);
-
-  /* If we don't have a texture, like during initialization, assume
-   * that we're ARGB32.
-   *
-   * If we are unredirected and we have no texture assume that we are
-   * not ARGB32 otherwise we wouldn't be unredirected in the first
-   * place. This prevents us from continually redirecting and
-   * unredirecting on every paint.
-   */
-  if (!texture)
-    return !meta_surface_actor_is_unredirected (self);
-
-  switch (cogl_texture_get_components (texture))
-    {
-    case COGL_TEXTURE_COMPONENTS_A:
-    case COGL_TEXTURE_COMPONENTS_RGBA:
-      return TRUE;
-    case COGL_TEXTURE_COMPONENTS_RG:
-    case COGL_TEXTURE_COMPONENTS_RGB:
-    case COGL_TEXTURE_COMPONENTS_DEPTH:
-      return FALSE;
-    default:
-      g_assert_not_reached ();
-      return FALSE;
-    }
-}
-
 gboolean
 meta_surface_actor_is_visible (MetaSurfaceActor *self)
 {
@@ -546,25 +520,6 @@ meta_surface_actor_set_frozen (MetaSurfaceActor *self,
     }
 }
 
-gboolean
-meta_surface_actor_should_unredirect (MetaSurfaceActor *self)
-{
-  return META_SURFACE_ACTOR_GET_CLASS (self)->should_unredirect (self);
-}
-
-void
-meta_surface_actor_set_unredirected (MetaSurfaceActor *self,
-                                     gboolean          unredirected)
-{
-  META_SURFACE_ACTOR_GET_CLASS (self)->set_unredirected (self, unredirected);
-}
-
-gboolean
-meta_surface_actor_is_unredirected (MetaSurfaceActor *self)
-{
-  return META_SURFACE_ACTOR_GET_CLASS (self)->is_unredirected (self);
-}
-
 MetaWindow *
 meta_surface_actor_get_window (MetaSurfaceActor *self)
 {
diff --git a/src/compositor/meta-surface-actor.h b/src/compositor/meta-surface-actor.h
index 1cffcf99c..ddf394155 100644
--- a/src/compositor/meta-surface-actor.h
+++ b/src/compositor/meta-surface-actor.h
@@ -26,11 +26,7 @@ struct _MetaSurfaceActorClass
                                   int x, int y, int width, int height);
   void     (* pre_paint)         (MetaSurfaceActor *actor);
   gboolean (* is_visible)        (MetaSurfaceActor *actor);
-
-  gboolean (* should_unredirect) (MetaSurfaceActor *actor);
-  void     (* set_unredirected)  (MetaSurfaceActor *actor,
-                                  gboolean          unredirected);
-  gboolean (* is_unredirected)   (MetaSurfaceActor *actor);
+  gboolean (* is_opaque)         (MetaSurfaceActor *actor);
 
   MetaWindow *(* get_window)      (MetaSurfaceActor *actor);
 };
@@ -52,17 +48,12 @@ cairo_region_t * meta_surface_actor_get_opaque_region (MetaSurfaceActor *self);
 void meta_surface_actor_process_damage (MetaSurfaceActor *actor,
                                         int x, int y, int width, int height);
 void meta_surface_actor_pre_paint (MetaSurfaceActor *actor);
-gboolean meta_surface_actor_is_argb32 (MetaSurfaceActor *actor);
 gboolean meta_surface_actor_is_visible (MetaSurfaceActor *actor);
+gboolean meta_surface_actor_is_opaque (MetaSurfaceActor *actor);
 
 void meta_surface_actor_set_frozen (MetaSurfaceActor *actor,
                                     gboolean          frozen);
 
-gboolean meta_surface_actor_should_unredirect (MetaSurfaceActor *actor);
-void meta_surface_actor_set_unredirected (MetaSurfaceActor *actor,
-                                          gboolean          unredirected);
-gboolean meta_surface_actor_is_unredirected (MetaSurfaceActor *actor);
-
 void meta_surface_actor_set_transform (MetaSurfaceActor     *self,
                                        MetaMonitorTransform  transform);
 void meta_surface_actor_set_viewport_src_rect (MetaSurfaceActor  *self,
diff --git a/src/compositor/meta-window-actor-private.h b/src/compositor/meta-window-actor-private.h
index a36217f98..1b60db025 100644
--- a/src/compositor/meta-window-actor-private.h
+++ b/src/compositor/meta-window-actor-private.h
@@ -46,30 +46,17 @@ void meta_window_actor_size_change   (MetaWindowActor *self,
                                       MetaRectangle   *old_frame_rect,
                                       MetaRectangle   *old_buffer_rect);
 
-void meta_window_actor_process_x11_damage (MetaWindowActor    *self,
-                                           XDamageNotifyEvent *event);
-
 void meta_window_actor_pre_paint      (MetaWindowActor    *self);
 void meta_window_actor_post_paint     (MetaWindowActor    *self);
 void meta_window_actor_frame_complete (MetaWindowActor    *self,
                                        ClutterFrameInfo   *frame_info,
                                        gint64              presentation_time);
 
-void meta_window_actor_invalidate_shadow (MetaWindowActor *self);
-
-void meta_window_actor_get_shape_bounds (MetaWindowActor       *self,
-                                          cairo_rectangle_int_t *bounds);
-
-gboolean meta_window_actor_should_unredirect   (MetaWindowActor *self);
-void     meta_window_actor_set_unredirected    (MetaWindowActor *self,
-                                                gboolean         unredirected);
-
 gboolean meta_window_actor_effect_in_progress  (MetaWindowActor *self);
 
 MetaWindowActorChanges meta_window_actor_sync_actor_geometry (MetaWindowActor *self,
                                                               gboolean         did_placement);
 
-void     meta_window_actor_update_shape        (MetaWindowActor *self);
 void     meta_window_actor_update_opacity      (MetaWindowActor *self);
 void     meta_window_actor_mapped              (MetaWindowActor *self);
 void     meta_window_actor_unmapped            (MetaWindowActor *self);
@@ -95,4 +82,8 @@ int meta_window_actor_get_geometry_scale (MetaWindowActor *window_actor);
 
 void meta_window_actor_notify_damaged (MetaWindowActor *window_actor);
 
+gboolean meta_window_actor_is_frozen (MetaWindowActor *self);
+
+gboolean meta_window_actor_is_opaque (MetaWindowActor *self);
+
 #endif /* META_WINDOW_ACTOR_PRIVATE_H */
diff --git a/src/compositor/meta-window-actor-wayland.c b/src/compositor/meta-window-actor-wayland.c
index e251e3a27..544d50564 100644
--- a/src/compositor/meta-window-actor-wayland.c
+++ b/src/compositor/meta-window-actor-wayland.c
@@ -109,6 +109,8 @@ meta_window_actor_wayland_assign_surface_actor (MetaWindowActor  *actor,
   MetaWindowActorClass *parent_class =
     META_WINDOW_ACTOR_CLASS (meta_window_actor_wayland_parent_class);
 
+  g_warn_if_fail (!meta_window_actor_get_surface (actor));
+
   parent_class->assign_surface_actor (actor, surface_actor);
 
   meta_window_actor_wayland_rebuild_surface_tree (actor);
@@ -142,10 +144,36 @@ meta_window_actor_wayland_queue_destroy (MetaWindowActor *actor)
 {
 }
 
+static gboolean
+meta_window_actor_wayland_get_paint_volume (ClutterActor       *actor,
+                                            ClutterPaintVolume *volume)
+{
+  MetaSurfaceActor *surface;
+
+  surface = meta_window_actor_get_surface (META_WINDOW_ACTOR (actor));
+  if (surface)
+    {
+      ClutterActor *surface_actor = CLUTTER_ACTOR (surface);
+      const ClutterPaintVolume *child_volume;
+
+      child_volume = clutter_actor_get_transformed_paint_volume (surface_actor,
+                                                                 actor);
+      if (!child_volume)
+        return FALSE;
+
+      clutter_paint_volume_union (volume, child_volume);
+    }
+
+  return TRUE;
+}
+
 static void
 meta_window_actor_wayland_class_init (MetaWindowActorWaylandClass *klass)
 {
   MetaWindowActorClass *window_actor_class = META_WINDOW_ACTOR_CLASS (klass);
+  ClutterActorClass *actor_class = CLUTTER_ACTOR_CLASS (klass);
+
+  actor_class->get_paint_volume = meta_window_actor_wayland_get_paint_volume;
 
   window_actor_class->assign_surface_actor = meta_window_actor_wayland_assign_surface_actor;
   window_actor_class->frame_complete = meta_window_actor_wayland_frame_complete;
diff --git a/src/compositor/meta-window-actor-x11.c b/src/compositor/meta-window-actor-x11.c
index 7902d7a1c..8bf65351f 100644
--- a/src/compositor/meta-window-actor-x11.c
+++ b/src/compositor/meta-window-actor-x11.c
@@ -26,14 +26,27 @@
 
 #include "backends/meta-logical-monitor.h"
 #include "compositor/compositor-private.h"
+#include "compositor/meta-cullable.h"
+#include "compositor/meta-shaped-texture-private.h"
 #include "compositor/meta-surface-actor.h"
+#include "compositor/meta-surface-actor-x11.h"
+#include "compositor/region-utils.h"
+#include "core/frame.h"
 #include "core/window-private.h"
 #include "meta/compositor.h"
+#include "meta/meta-enum-types.h"
+#include "meta/meta-shadow-factory.h"
 #include "meta/meta-window-actor.h"
 #include "meta/meta-x11-errors.h"
 #include "meta/window.h"
 #include "x11/meta-x11-display-private.h"
 
+enum
+{
+  PROP_SHADOW_MODE = 1,
+  PROP_SHADOW_CLASS
+};
+
 struct _MetaWindowActorX11
 {
   MetaWindowActor parent;
@@ -45,14 +58,53 @@ struct _MetaWindowActorX11
   int64_t frame_drawn_time;
 
   gulong repaint_scheduled_id;
+  gulong size_changed_id;
 
   /* If set, the client needs to be sent a _NET_WM_FRAME_DRAWN
    * client message for one or more messages in ->frames */
   gboolean needs_frame_drawn;
   gboolean repaint_scheduled;
+
+  /*
+   * MetaShadowFactory only caches shadows that are actually in use;
+   * to avoid unnecessary recomputation we do two things: 1) we store
+   * both a focused and unfocused shadow for the window. If the window
+   * doesn't have different focused and unfocused shadow parameters,
+   * these will be the same. 2) when the shadow potentially changes we
+   * don't immediately unreference the old shadow, we just flag it as
+   * dirty and recompute it when we next need it (recompute_focused_shadow,
+   * recompute_unfocused_shadow.) Because of our extraction of
+   * size-invariant window shape, we'll often find that the new shadow
+   * is the same as the old shadow.
+   */
+  MetaShadow *focused_shadow;
+  MetaShadow *unfocused_shadow;
+
+  /* A region that matches the shape of the window, including frame bounds */
+  cairo_region_t *shape_region;
+  /* The region we should clip to when painting the shadow */
+  cairo_region_t *shadow_clip;
+
+  /* Extracted size-invariant shape used for shadows */
+  MetaWindowShape *shadow_shape;
+  char *shadow_class;
+
+  MetaShadowFactory *shadow_factory;
+  gulong shadow_factory_changed_handler_id;
+
+  MetaShadowMode shadow_mode;
+
+  gboolean needs_reshape;
+  gboolean recompute_focused_shadow;
+  gboolean recompute_unfocused_shadow;
 };
 
-G_DEFINE_TYPE (MetaWindowActorX11, meta_window_actor_x11, META_TYPE_WINDOW_ACTOR)
+static MetaCullableInterface *cullable_parent_iface;
+
+static void cullable_iface_init (MetaCullableInterface *iface);
+
+G_DEFINE_TYPE_WITH_CODE (MetaWindowActorX11, meta_window_actor_x11, META_TYPE_WINDOW_ACTOR,
+                         G_IMPLEMENT_INTERFACE (META_TYPE_CULLABLE, cullable_iface_init))
 
 /* Each time the application updates the sync request counter to a new even value
  * value, we queue a frame into the windows list of frames. Once we're painting
@@ -328,6 +380,15 @@ meta_window_actor_x11_frame_complete (MetaWindowActor  *actor,
     }
 }
 
+static void
+surface_size_changed (MetaSurfaceActor *actor,
+                      gpointer          user_data)
+{
+  MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (user_data);
+
+  meta_window_actor_x11_update_shape (actor_x11);
+}
+
 static void
 meta_window_actor_x11_assign_surface_actor (MetaWindowActor  *actor,
                                             MetaSurfaceActor *surface_actor)
@@ -335,12 +396,29 @@ meta_window_actor_x11_assign_surface_actor (MetaWindowActor  *actor,
   MetaWindowActorClass *parent_class =
     META_WINDOW_ACTOR_CLASS (meta_window_actor_x11_parent_class);
   MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (actor);
+  MetaSurfaceActor *prev_surface_actor;
+
+  prev_surface_actor = meta_window_actor_get_surface (actor);
+  if (prev_surface_actor)
+    {
+      g_warn_if_fail (meta_is_wayland_compositor ());
+
+      g_clear_signal_handler (&actor_x11->size_changed_id, prev_surface_actor);
+      clutter_actor_remove_child (CLUTTER_ACTOR (actor),
+                                  CLUTTER_ACTOR (prev_surface_actor));
+    }
 
   parent_class->assign_surface_actor (actor, surface_actor);
 
   clutter_actor_add_child (CLUTTER_ACTOR (actor),
                            CLUTTER_ACTOR (surface_actor));
 
+  meta_window_actor_x11_update_shape (actor_x11);
+
+  actor_x11->size_changed_id =
+    g_signal_connect (surface_actor, "size-changed",
+                      G_CALLBACK (surface_size_changed),
+                      actor_x11);
   actor_x11->repaint_scheduled_id =
     g_signal_connect (surface_actor, "repaint-scheduled",
                       G_CALLBACK (surface_repaint_scheduled),
@@ -406,14 +484,728 @@ meta_window_actor_x11_queue_frame_drawn (MetaWindowActor *actor,
     }
 }
 
+static gboolean
+has_shadow (MetaWindowActorX11 *actor_x11)
+{
+  MetaWindow *window =
+    meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+
+  if (actor_x11->shadow_mode == META_SHADOW_MODE_FORCED_OFF)
+    return FALSE;
+  if (actor_x11->shadow_mode == META_SHADOW_MODE_FORCED_ON)
+    return TRUE;
+
+  /* Leaving out shadows for maximized and fullscreen windows is an effeciency
+   * win and also prevents the unsightly effect of the shadow of maximized
+   * window appearing on an adjacent window */
+  if ((meta_window_get_maximized (window) == META_MAXIMIZE_BOTH) ||
+      meta_window_is_fullscreen (window))
+    return FALSE;
+
+  /*
+   * If we have two snap-tiled windows, we don't want the shadow to obstruct
+   * the other window.
+   */
+  if (meta_window_get_tile_match (window))
+    return FALSE;
+
+  /*
+   * Always put a shadow around windows with a frame - This should override
+   * the restriction about not putting a shadow around ARGB windows.
+   */
+  if (meta_window_get_frame (window))
+    return TRUE;
+
+  /*
+   * Do not add shadows to non-opaque (ARGB32) windows, as we can't easily
+   * generate shadows for them.
+   */
+  if (!meta_window_actor_is_opaque (META_WINDOW_ACTOR (actor_x11)))
+    return FALSE;
+
+  /*
+   * If a window specifies that it has custom frame extents, that likely
+   * means that it is drawing a shadow itself. Don't draw our own.
+   */
+  if (window->has_custom_frame_extents)
+    return FALSE;
+
+  /*
+   * Generate shadows for all other windows.
+   */
+  return TRUE;
+}
+
+gboolean
+meta_window_actor_x11_should_unredirect (MetaWindowActorX11 *actor_x11)
+{
+  MetaSurfaceActor *surface;
+  MetaSurfaceActorX11 *surface_x11;
+
+  if (meta_window_actor_is_destroyed (META_WINDOW_ACTOR (actor_x11)))
+    return FALSE;
+
+  surface = meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+  if (!surface)
+    return FALSE;
+
+  if (!META_IS_SURFACE_ACTOR_X11 (surface))
+    return FALSE;
+
+  surface_x11 = META_SURFACE_ACTOR_X11 (surface);
+  return meta_surface_actor_x11_should_unredirect (surface_x11);
+}
+
+void
+meta_window_actor_x11_set_unredirected (MetaWindowActorX11 *actor_x11,
+                                        gboolean            unredirected)
+{
+  MetaSurfaceActor *surface;
+  MetaSurfaceActorX11 *surface_x11;
+
+  surface = meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+  g_assert (surface);
+
+  g_return_if_fail (META_IS_SURFACE_ACTOR_X11 (surface));
+
+  surface_x11 = META_SURFACE_ACTOR_X11 (surface);
+  meta_surface_actor_x11_set_unredirected (surface_x11, unredirected);
+}
+
+static const char *
+get_shadow_class (MetaWindowActorX11 *actor_x11)
+{
+  if (actor_x11->shadow_class)
+    {
+      return actor_x11->shadow_class;
+    }
+  else
+    {
+      MetaWindow *window =
+        meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+      MetaWindowType window_type;
+
+      window_type = meta_window_get_window_type (window);
+      switch (window_type)
+        {
+        case META_WINDOW_DROPDOWN_MENU:
+        case META_WINDOW_COMBO:
+          return "dropdown-menu";
+        case META_WINDOW_POPUP_MENU:
+          return "popup-menu";
+        default:
+          {
+            MetaFrameType frame_type;
+
+            frame_type = meta_window_get_frame_type (window);
+            return meta_frame_type_to_string (frame_type);
+          }
+        }
+    }
+}
+
+static void
+get_shadow_params (MetaWindowActorX11 *actor_x11,
+                   gboolean            appears_focused,
+                   MetaShadowParams   *params)
+{
+  const char *shadow_class = get_shadow_class (actor_x11);
+
+  meta_shadow_factory_get_params (actor_x11->shadow_factory,
+                                  shadow_class, appears_focused,
+                                  params);
+}
+
+static void
+get_shape_bounds (MetaWindowActorX11    *actor_x11,
+                  cairo_rectangle_int_t *bounds)
+{
+  cairo_region_get_extents (actor_x11->shape_region, bounds);
+}
+
+static void
+get_shadow_bounds (MetaWindowActorX11    *actor_x11,
+                   gboolean               appears_focused,
+                   cairo_rectangle_int_t *bounds)
+{
+  MetaShadow *shadow;
+  cairo_rectangle_int_t shape_bounds;
+  MetaShadowParams params;
+
+  shadow = appears_focused ? actor_x11->focused_shadow
+                           : actor_x11->unfocused_shadow;
+
+  get_shape_bounds (actor_x11, &shape_bounds);
+  get_shadow_params (actor_x11, appears_focused, &params);
+
+  meta_shadow_get_bounds (shadow,
+                          params.x_offset + shape_bounds.x,
+                          params.y_offset + shape_bounds.y,
+                          shape_bounds.width,
+                          shape_bounds.height,
+                          bounds);
+}
+
+/* If we have an ARGB32 window that we decorate with a frame, it's
+ * probably something like a translucent terminal - something where
+ * the alpha channel represents transparency rather than a shape.  We
+ * don't want to show the shadow through the translucent areas since
+ * the shadow is wrong for translucent windows (it should be
+ * translucent itself and colored), and not only that, will /look/
+ * horribly wrong - a misplaced big black blob. As a hack, what we
+ * want to do is just draw the shadow as normal outside the frame, and
+ * inside the frame draw no shadow.  This is also not even close to
+ * the right result, but looks OK. We also apply this approach to
+ * windows set to be partially translucent with _NET_WM_WINDOW_OPACITY.
+ */
+static gboolean
+clip_shadow_under_window (MetaWindowActorX11 *actor_x11)
+{
+  MetaWindow *window =
+    meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+
+  if (window->frame)
+    return TRUE;
+
+  return meta_window_actor_is_opaque (META_WINDOW_ACTOR (actor_x11));
+}
+
+/**
+ * set_clip_region_beneath:
+ * @actor_x11: a #MetaWindowActorX11
+ * @clip_region: the region of the screen that isn't completely
+ *  obscured beneath the main window texture.
+ *
+ * Provides a hint as to what areas need to be drawn *beneath*
+ * the main window texture.  This is the relevant clip region
+ * when drawing the shadow, properly accounting for areas of the
+ * shadow hid by the window itself. This will be set before painting
+ * then unset afterwards.
+ */
+static void
+set_clip_region_beneath (MetaWindowActorX11 *actor_x11,
+                         cairo_region_t     *beneath_region)
+{
+  MetaWindow *window;
+  gboolean appears_focused;
+
+  window = meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+  appears_focused = meta_window_appears_focused (window);
+  if (appears_focused ? actor_x11->focused_shadow : actor_x11->unfocused_shadow)
+    {
+      g_clear_pointer (&actor_x11->shadow_clip, cairo_region_destroy);
+
+      if (beneath_region)
+        {
+          actor_x11->shadow_clip = cairo_region_copy (beneath_region);
+
+          if (clip_shadow_under_window (actor_x11))
+            {
+              cairo_region_t *frame_bounds;
+
+              frame_bounds = meta_window_get_frame_bounds (window);
+              if (frame_bounds)
+                cairo_region_subtract (actor_x11->shadow_clip, frame_bounds);
+            }
+        }
+      else
+        {
+          actor_x11->shadow_clip = NULL;
+        }
+    }
+}
+
+static void
+check_needs_shadow (MetaWindowActorX11 *actor_x11)
+{
+  MetaWindow *window =
+    meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+  MetaShadow *old_shadow = NULL;
+  MetaShadow **shadow_location;
+  gboolean recompute_shadow;
+  gboolean should_have_shadow;
+  gboolean appears_focused;
+
+  /* Calling has_shadow() here at every pre-paint is cheap
+   * and avoids the need to explicitly handle window type changes, which
+   * we would do if tried to keep track of when we might be adding or removing
+   * a shadow more explicitly. We only keep track of changes to the *shape* of
+   * the shadow with actor_x11->recompute_shadow.
+   */
+
+  should_have_shadow = has_shadow (actor_x11);
+  appears_focused = meta_window_appears_focused (window);
+
+  if (appears_focused)
+    {
+      recompute_shadow = actor_x11->recompute_focused_shadow;
+      actor_x11->recompute_focused_shadow = FALSE;
+      shadow_location = &actor_x11->focused_shadow;
+    }
+  else
+    {
+      recompute_shadow = actor_x11->recompute_unfocused_shadow;
+      actor_x11->recompute_unfocused_shadow = FALSE;
+      shadow_location = &actor_x11->unfocused_shadow;
+    }
+
+  if (!should_have_shadow || recompute_shadow)
+    {
+      if (*shadow_location != NULL)
+        {
+          old_shadow = *shadow_location;
+          *shadow_location = NULL;
+        }
+    }
+
+  if (!*shadow_location && should_have_shadow)
+    {
+      MetaShadowFactory *factory = actor_x11->shadow_factory;
+      const char *shadow_class = get_shadow_class (actor_x11);
+      cairo_rectangle_int_t shape_bounds;
+
+      if (!actor_x11->shadow_shape)
+        {
+          actor_x11->shadow_shape =
+            meta_window_shape_new (actor_x11->shape_region);
+        }
+
+      get_shape_bounds (actor_x11, &shape_bounds);
+      *shadow_location =
+        meta_shadow_factory_get_shadow (factory,
+                                        actor_x11->shadow_shape,
+                                        shape_bounds.width, shape_bounds.height,
+                                        shadow_class, appears_focused);
+    }
+
+  if (old_shadow)
+    meta_shadow_unref (old_shadow);
+}
+
+void
+meta_window_actor_x11_process_damage (MetaWindowActorX11 *actor_x11,
+                                      XDamageNotifyEvent *event)
+{
+  MetaSurfaceActor *surface;
+
+  surface = meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+  if (surface)
+    meta_surface_actor_process_damage (surface,
+                                       event->area.x,
+                                       event->area.y,
+                                       event->area.width,
+                                       event->area.height);
+
+  meta_window_actor_notify_damaged (META_WINDOW_ACTOR (actor_x11));
+}
+
+static cairo_region_t *
+scan_visible_region (guchar         *mask_data,
+                     int             stride,
+                     cairo_region_t *scan_area)
+{
+  int i, n_rects = cairo_region_num_rectangles (scan_area);
+  MetaRegionBuilder builder;
+
+  meta_region_builder_init (&builder);
+
+  for (i = 0; i < n_rects; i++)
+    {
+      int x, y;
+      cairo_rectangle_int_t rect;
+
+      cairo_region_get_rectangle (scan_area, i, &rect);
+
+      for (y = rect.y; y < (rect.y + rect.height); y++)
+        {
+          for (x = rect.x; x < (rect.x + rect.width); x++)
+            {
+              int x2 = x;
+              while (mask_data[y * stride + x2] == 255 && x2 < (rect.x + rect.width))
+                x2++;
+
+              if (x2 > x)
+                {
+                  meta_region_builder_add_rectangle (&builder, x, y, x2 - x, 1);
+                  x = x2;
+                }
+            }
+        }
+    }
+
+  return meta_region_builder_finish (&builder);
+}
+
+static void
+build_and_scan_frame_mask (MetaWindowActorX11    *actor_x11,
+                           cairo_rectangle_int_t *client_area,
+                           cairo_region_t        *shape_region)
+{
+  ClutterBackend *backend = clutter_get_default_backend ();
+  MetaWindow *window =
+    meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+  CoglContext *ctx = clutter_backend_get_cogl_context (backend);
+  MetaSurfaceActor *surface =
+    meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+  uint8_t *mask_data;
+  unsigned int tex_width, tex_height;
+  MetaShapedTexture *stex;
+  CoglTexture *paint_tex;
+  CoglTexture2D *mask_texture;
+  int stride;
+  cairo_t *cr;
+  cairo_surface_t *image;
+  GError *error = NULL;
+
+  stex = meta_surface_actor_get_texture (surface);
+  g_return_if_fail (stex);
+
+  meta_shaped_texture_set_mask_texture (stex, NULL);
+
+  paint_tex = meta_shaped_texture_get_texture (stex);
+  if (paint_tex == NULL)
+    return;
+
+  tex_width = cogl_texture_get_width (paint_tex);
+  tex_height = cogl_texture_get_height (paint_tex);
+
+  stride = cairo_format_stride_for_width (CAIRO_FORMAT_A8, tex_width);
+
+  /* Create data for an empty image */
+  mask_data = g_malloc0 (stride * tex_height);
+
+  image = cairo_image_surface_create_for_data (mask_data,
+                                               CAIRO_FORMAT_A8,
+                                               tex_width,
+                                               tex_height,
+                                               stride);
+  cr = cairo_create (image);
+
+  gdk_cairo_region (cr, shape_region);
+  cairo_fill (cr);
+
+  if (window->frame)
+    {
+      cairo_region_t *frame_paint_region, *scanned_region;
+      cairo_rectangle_int_t rect = { 0, 0, tex_width, tex_height };
+
+      /* Make sure we don't paint the frame over the client window. */
+      frame_paint_region = cairo_region_create_rectangle (&rect);
+      cairo_region_subtract_rectangle (frame_paint_region, client_area);
+
+      gdk_cairo_region (cr, frame_paint_region);
+      cairo_clip (cr);
+
+      meta_frame_get_mask (window->frame, cr);
+
+      cairo_surface_flush (image);
+      scanned_region = scan_visible_region (mask_data, stride, frame_paint_region);
+      cairo_region_union (shape_region, scanned_region);
+      cairo_region_destroy (scanned_region);
+      cairo_region_destroy (frame_paint_region);
+    }
+
+  cairo_destroy (cr);
+  cairo_surface_destroy (image);
+
+  mask_texture = cogl_texture_2d_new_from_data (ctx, tex_width, tex_height,
+                                                COGL_PIXEL_FORMAT_A_8,
+                                                stride, mask_data, &error);
+
+  if (error)
+    {
+      g_warning ("Failed to allocate mask texture: %s", error->message);
+      g_error_free (error);
+    }
+
+  if (mask_texture)
+    {
+      meta_shaped_texture_set_mask_texture (stex, COGL_TEXTURE (mask_texture));
+      cogl_object_unref (mask_texture);
+    }
+  else
+    {
+      meta_shaped_texture_set_mask_texture (stex, NULL);
+    }
+
+  g_free (mask_data);
+}
+
+static void
+invalidate_shadow (MetaWindowActorX11 *actor_x11)
+{
+  actor_x11->recompute_focused_shadow = TRUE;
+  actor_x11->recompute_unfocused_shadow = TRUE;
+
+  if (meta_window_actor_is_frozen (META_WINDOW_ACTOR (actor_x11)))
+    return;
+
+  clutter_actor_queue_redraw (CLUTTER_ACTOR (actor_x11));
+}
+
+static void
+update_shape_region (MetaWindowActorX11 *actor_x11)
+{
+  MetaWindow *window =
+    meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+  cairo_region_t *region = NULL;
+  cairo_rectangle_int_t client_area;
+
+  meta_window_get_client_area_rect (window, &client_area);
+
+  if (window->frame && window->shape_region)
+    {
+      region = cairo_region_copy (window->shape_region);
+      cairo_region_translate (region, client_area.x, client_area.y);
+    }
+  else if (window->shape_region != NULL)
+    {
+      region = cairo_region_reference (window->shape_region);
+    }
+  else
+    {
+      /* If we don't have a shape on the server, that means that
+       * we have an implicit shape of one rectangle covering the
+       * entire window. */
+      region = cairo_region_create_rectangle (&client_area);
+    }
+
+  if (window->shape_region || window->frame)
+    build_and_scan_frame_mask (actor_x11, &client_area, region);
+
+  g_clear_pointer (&actor_x11->shape_region, cairo_region_destroy);
+  actor_x11->shape_region = region;
+
+  g_clear_pointer (&actor_x11->shadow_shape, meta_window_shape_unref);
+
+  invalidate_shadow (actor_x11);
+}
+
+static void
+update_input_region (MetaWindowActorX11 *actor_x11)
+{
+  MetaWindow *window =
+    meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+  MetaSurfaceActor *surface =
+    meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+  cairo_region_t *region;
+
+  if (window->shape_region && window->input_region)
+    {
+      region = cairo_region_copy (window->shape_region);
+      cairo_region_intersect (region, window->input_region);
+    }
+  else if (window->shape_region)
+    {
+      region = cairo_region_reference (window->shape_region);
+    }
+  else if (window->input_region)
+    {
+      region = cairo_region_reference (window->input_region);
+    }
+  else
+    {
+      region = NULL;
+    }
+
+  meta_surface_actor_set_input_region (surface, region);
+  cairo_region_destroy (region);
+}
+
+static gboolean
+is_actor_maybe_transparent (MetaWindowActorX11 *actor_x11)
+{
+  MetaSurfaceActor *surface;
+  MetaShapedTexture *stex;
+
+  surface = meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+  if (!surface)
+    return TRUE;
+
+  if (META_IS_SURFACE_ACTOR_X11 (surface) &&
+      meta_surface_actor_x11_is_unredirected (META_SURFACE_ACTOR_X11 (surface)))
+    return FALSE;
+
+  stex = meta_surface_actor_get_texture (surface);
+  if (!meta_shaped_texture_has_alpha (stex))
+    return FALSE;
+
+  return TRUE;
+}
+
+static void
+update_opaque_region (MetaWindowActorX11 *actor_x11)
+{
+  MetaWindow *window =
+    meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+  gboolean is_maybe_transparent;
+  cairo_region_t *opaque_region;
+  MetaSurfaceActor *surface;
+
+  is_maybe_transparent = is_actor_maybe_transparent (actor_x11);
+  if (is_maybe_transparent && window->opaque_region)
+    {
+      cairo_rectangle_int_t client_area;
+
+      meta_window_get_client_area_rect (window, &client_area);
+
+      /* The opaque region is defined to be a part of the
+       * window which ARGB32 will always paint with opaque
+       * pixels. For these regions, we want to avoid painting
+       * windows and shadows beneath them.
+       *
+       * If the client gives bad coordinates where it does not
+       * fully paint, the behavior is defined by the specification
+       * to be undefined, and considered a client bug. In mutter's
+       * case, graphical glitches will occur.
+       */
+      opaque_region = cairo_region_copy (window->opaque_region);
+      cairo_region_translate (opaque_region, client_area.x, client_area.y);
+      cairo_region_intersect (opaque_region, actor_x11->shape_region);
+    }
+  else if (is_maybe_transparent)
+    {
+      opaque_region = NULL;
+    }
+  else
+    {
+      opaque_region = cairo_region_reference (actor_x11->shape_region);
+    }
+
+  surface = meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+  meta_surface_actor_set_opaque_region (surface, opaque_region);
+  cairo_region_destroy (opaque_region);
+}
+
+static void
+check_needs_reshape (MetaWindowActorX11 *actor_x11)
+{
+  if (!actor_x11->needs_reshape)
+    return;
+
+  update_shape_region (actor_x11);
+  update_input_region (actor_x11);
+  update_opaque_region (actor_x11);
+
+  actor_x11->needs_reshape = FALSE;
+}
+
+void
+meta_window_actor_x11_update_shape (MetaWindowActorX11 *actor_x11)
+{
+  MetaSurfaceActor *surface =
+    meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+
+  actor_x11->needs_reshape = TRUE;
+
+  if (meta_window_actor_is_frozen (META_WINDOW_ACTOR (actor_x11)))
+    return;
+
+  clutter_actor_queue_redraw (CLUTTER_ACTOR (surface));
+}
+
+static void
+handle_updates (MetaWindowActorX11 *actor_x11)
+{
+  MetaSurfaceActor *surface =
+    meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+
+  if (meta_window_actor_is_frozen (META_WINDOW_ACTOR (actor_x11)))
+    {
+      /* The window is frozen due to a pending animation: we'll wait until
+       * the animation finishes to reshape and repair the window */
+      return;
+    }
+
+  if (META_IS_SURFACE_ACTOR_X11 (surface) &&
+      meta_surface_actor_x11_is_unredirected (META_SURFACE_ACTOR_X11 (surface)))
+    return;
+
+  meta_surface_actor_pre_paint (surface);
+
+  if (!meta_surface_actor_is_visible (surface))
+    return;
+
+  check_needs_reshape (actor_x11);
+  check_needs_shadow (actor_x11);
+}
+
 static void
 meta_window_actor_x11_pre_paint (MetaWindowActor *actor)
 {
   MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (actor);
 
+  handle_updates (actor_x11);
+
   assign_frame_counter_to_frames (actor_x11);
 }
 
+static void
+meta_window_actor_x11_paint (ClutterActor *actor)
+{
+  MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (actor);
+  MetaWindow *window;
+  gboolean appears_focused;
+  MetaShadow *shadow;
+
+ /* This window got damage when obscured; we set up a timer
+  * to send frame completion events, but since we're drawing
+  * the window now (for some other reason) cancel the timer
+  * and send the completion events normally */
+  if (actor_x11->send_frame_messages_timer != 0)
+    {
+      remove_frame_messages_timer (actor_x11);
+      assign_frame_counter_to_frames (actor_x11);
+    }
+
+  window = meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+  appears_focused = meta_window_appears_focused (window);
+  shadow = appears_focused ? actor_x11->focused_shadow
+                           : actor_x11->unfocused_shadow;
+
+  if (shadow)
+    {
+      MetaShadowParams params;
+      cairo_rectangle_int_t shape_bounds;
+      cairo_region_t *clip = actor_x11->shadow_clip;
+
+      get_shape_bounds (actor_x11, &shape_bounds);
+      get_shadow_params (actor_x11, appears_focused, &params);
+
+      /* The frame bounds are already subtracted from actor_x11->shadow_clip
+       * if that exists.
+       */
+      if (!clip && clip_shadow_under_window (actor_x11))
+        {
+          cairo_region_t *frame_bounds;
+          cairo_rectangle_int_t bounds;
+
+          get_shadow_bounds (actor_x11, appears_focused, &bounds);
+          clip = cairo_region_create_rectangle (&bounds);
+
+          frame_bounds = meta_window_get_frame_bounds (window);
+          if (frame_bounds)
+            cairo_region_subtract (clip, frame_bounds);
+        }
+
+      meta_shadow_paint (shadow,
+                         cogl_get_draw_framebuffer (),
+                         params.x_offset + shape_bounds.x,
+                         params.y_offset + shape_bounds.y,
+                         shape_bounds.width,
+                         shape_bounds.height,
+                         (clutter_actor_get_paint_opacity (actor) *
+                          params.opacity * window->opacity) / (255 * 255),
+                         clip,
+                         clip_shadow_under_window (actor_x11));
+
+      if (clip && clip != actor_x11->shadow_clip)
+        cairo_region_destroy (clip);
+    }
+
+  CLUTTER_ACTOR_CLASS (meta_window_actor_x11_parent_class)->paint (actor);
+}
+
 static void
 meta_window_actor_x11_post_paint (MetaWindowActor *actor)
 {
@@ -445,6 +1237,59 @@ meta_window_actor_x11_post_paint (MetaWindowActor *actor)
     }
 }
 
+static gboolean
+meta_window_actor_x11_get_paint_volume (ClutterActor       *actor,
+                                        ClutterPaintVolume *volume)
+{
+  MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (actor);
+  MetaWindow *window;
+  gboolean appears_focused;
+  MetaSurfaceActor *surface;
+
+  /* The paint volume is computed before paint functions are called
+   * so our bounds might not be updated yet. Force an update. */
+  handle_updates (actor_x11);
+
+  window = meta_window_actor_get_meta_window (META_WINDOW_ACTOR (actor_x11));
+  appears_focused = meta_window_appears_focused (window);
+  if (appears_focused ? actor_x11->focused_shadow : actor_x11->unfocused_shadow)
+    {
+      cairo_rectangle_int_t shadow_bounds;
+      ClutterActorBox shadow_box;
+
+      /* We could compute an full clip region as we do for the window
+       * texture, but the shadow is relatively cheap to draw, and
+       * a little more complex to clip, so we just catch the case where
+       * the shadow is completely obscured and doesn't need to be drawn
+       * at all.
+       */
+
+      get_shadow_bounds (actor_x11, appears_focused, &shadow_bounds);
+      shadow_box.x1 = shadow_bounds.x;
+      shadow_box.x2 = shadow_bounds.x + shadow_bounds.width;
+      shadow_box.y1 = shadow_bounds.y;
+      shadow_box.y2 = shadow_bounds.y + shadow_bounds.height;
+
+      clutter_paint_volume_union_box (volume, &shadow_box);
+    }
+
+  surface = meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
+  if (surface)
+    {
+      ClutterActor *surface_actor = CLUTTER_ACTOR (surface);
+      const ClutterPaintVolume *child_volume;
+
+      child_volume = clutter_actor_get_transformed_paint_volume (surface_actor,
+                                                                 actor);
+      if (!child_volume)
+        return FALSE;
+
+      clutter_paint_volume_union (volume, child_volume);
+    }
+
+  return TRUE;
+}
+
 static void
 meta_window_actor_x11_queue_destroy (MetaWindowActor *actor)
 {
@@ -455,29 +1300,80 @@ meta_window_actor_x11_queue_destroy (MetaWindowActor *actor)
 }
 
 static void
-meta_window_actor_x11_paint (ClutterActor *actor)
+meta_window_actor_x11_set_property (GObject      *object,
+                                    guint         prop_id,
+                                    const GValue *value,
+                                    GParamSpec   *pspec)
 {
-  MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (actor);
+  MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (object);
 
- /* This window got damage when obscured; we set up a timer
-  * to send frame completion events, but since we're drawing
-  * the window now (for some other reason) cancel the timer
-  * and send the completion events normally */
-  if (actor_x11->send_frame_messages_timer != 0)
+  switch (prop_id)
     {
-      remove_frame_messages_timer (actor_x11);
-      assign_frame_counter_to_frames (actor_x11);
+    case PROP_SHADOW_MODE:
+      {
+        MetaShadowMode newv = g_value_get_enum (value);
+
+        if (newv == actor_x11->shadow_mode)
+          return;
+
+        actor_x11->shadow_mode = newv;
+
+        invalidate_shadow (actor_x11);
+      }
+      break;
+    case PROP_SHADOW_CLASS:
+      {
+        const char *newv = g_value_get_string (value);
+
+        if (g_strcmp0 (newv, actor_x11->shadow_class) == 0)
+          return;
+
+        g_free (actor_x11->shadow_class);
+        actor_x11->shadow_class = g_strdup (newv);
+
+        invalidate_shadow (actor_x11);
+      }
+      break;
+    default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
+      break;
     }
+}
 
-  CLUTTER_ACTOR_CLASS (meta_window_actor_x11_parent_class)->paint (actor);
+static void
+meta_window_actor_x11_get_property (GObject      *object,
+                                    guint         prop_id,
+                                    GValue       *value,
+                                    GParamSpec   *pspec)
+{
+  MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (object);
+
+  switch (prop_id)
+    {
+    case PROP_SHADOW_MODE:
+      g_value_set_enum (value, actor_x11->shadow_mode);
+      break;
+    case PROP_SHADOW_CLASS:
+      g_value_set_string (value, actor_x11->shadow_class);
+      break;
+    default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
+      break;
+    }
 }
 
 static void
 meta_window_actor_x11_constructed (GObject *object)
 {
-  MetaWindowActor *window_actor = META_WINDOW_ACTOR (object);
-  MetaWindow *window =
-    meta_window_actor_get_meta_window (window_actor);
+  MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (object);
+  MetaWindowActor *actor = META_WINDOW_ACTOR (actor_x11);
+  MetaWindow *window = meta_window_actor_get_meta_window (actor);
+
+  /*
+   * Start off with an empty shape region to maintain the invariant that it's
+   * always set.
+   */
+  actor_x11->shape_region = cairo_region_create ();
 
   G_OBJECT_CLASS (meta_window_actor_x11_parent_class)->constructed (object);
 
@@ -486,7 +1382,36 @@ meta_window_actor_x11_constructed (GObject *object)
    */
   if (window->extended_sync_request_counter &&
       !meta_window_updates_are_frozen (window))
-    meta_window_actor_queue_frame_drawn (window_actor, FALSE);
+    meta_window_actor_queue_frame_drawn (actor, FALSE);
+}
+
+static void
+meta_window_actor_x11_cull_out (MetaCullable   *cullable,
+                                cairo_region_t *unobscured_region,
+                                cairo_region_t *clip_region)
+{
+  cullable_parent_iface->cull_out (cullable, unobscured_region, clip_region);
+
+  set_clip_region_beneath (META_WINDOW_ACTOR_X11 (cullable), clip_region);
+}
+
+static void
+meta_window_actor_x11_reset_culling (MetaCullable *cullable)
+{
+  MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (cullable);
+
+  g_clear_pointer (&actor_x11->shadow_clip, cairo_region_destroy);
+
+  cullable_parent_iface->reset_culling (cullable);
+}
+
+static void
+cullable_iface_init (MetaCullableInterface *iface)
+{
+  cullable_parent_iface = g_type_interface_peek_parent (iface);
+
+  iface->cull_out = meta_window_actor_x11_cull_out;
+  iface->reset_culling = meta_window_actor_x11_reset_culling;
 }
 
 static void
@@ -495,12 +1420,26 @@ meta_window_actor_x11_dispose (GObject *object)
   MetaWindowActorX11 *actor_x11 = META_WINDOW_ACTOR_X11 (object);
   MetaSurfaceActor *surface_actor;
 
+  g_clear_signal_handler (&actor_x11->shadow_factory_changed_handler_id,
+                          actor_x11->shadow_factory);
+
   if (actor_x11->send_frame_messages_timer != 0)
     remove_frame_messages_timer (actor_x11);
 
   surface_actor = meta_window_actor_get_surface (META_WINDOW_ACTOR (actor_x11));
   if (surface_actor)
-    g_clear_signal_handler (&actor_x11->repaint_scheduled_id, surface_actor);
+    {
+      g_clear_signal_handler (&actor_x11->repaint_scheduled_id, surface_actor);
+      g_clear_signal_handler (&actor_x11->size_changed_id, surface_actor);
+    }
+
+  g_clear_pointer (&actor_x11->shape_region, cairo_region_destroy);
+  g_clear_pointer (&actor_x11->shadow_clip, cairo_region_destroy);
+
+  g_clear_pointer (&actor_x11->shadow_class, g_free);
+  g_clear_pointer (&actor_x11->focused_shadow, meta_shadow_unref);
+  g_clear_pointer (&actor_x11->unfocused_shadow, meta_shadow_unref);
+  g_clear_pointer (&actor_x11->shadow_shape, meta_window_shape_unref);
 
   G_OBJECT_CLASS (meta_window_actor_x11_parent_class)->dispose (object);
 }
@@ -521,6 +1460,7 @@ meta_window_actor_x11_class_init (MetaWindowActorX11Class *klass)
   GObjectClass *object_class = G_OBJECT_CLASS (klass);
   ClutterActorClass *actor_class = CLUTTER_ACTOR_CLASS (klass);
   MetaWindowActorClass *window_actor_class = META_WINDOW_ACTOR_CLASS (klass);
+  GParamSpec *pspec;
 
   window_actor_class->frame_complete = meta_window_actor_x11_frame_complete;
   window_actor_class->assign_surface_actor = meta_window_actor_x11_assign_surface_actor;
@@ -530,13 +1470,46 @@ meta_window_actor_x11_class_init (MetaWindowActorX11Class *klass)
   window_actor_class->queue_destroy = meta_window_actor_x11_queue_destroy;
 
   actor_class->paint = meta_window_actor_x11_paint;
+  actor_class->get_paint_volume = meta_window_actor_x11_get_paint_volume;
 
   object_class->constructed = meta_window_actor_x11_constructed;
+  object_class->set_property = meta_window_actor_x11_set_property;
+  object_class->get_property = meta_window_actor_x11_get_property;
   object_class->dispose = meta_window_actor_x11_dispose;
   object_class->finalize = meta_window_actor_x11_finalize;
+
+  pspec = g_param_spec_enum ("shadow-mode",
+                             "Shadow mode",
+                             "Decides when to paint shadows",
+                             META_TYPE_SHADOW_MODE,
+                             META_SHADOW_MODE_AUTO,
+                             G_PARAM_READWRITE);
+
+  g_object_class_install_property (object_class,
+                                   PROP_SHADOW_MODE,
+                                   pspec);
+
+  pspec = g_param_spec_string ("shadow-class",
+                               "Name of the shadow class for this window.",
+                               "NULL means to use the default shadow class for this window type",
+                               NULL,
+                               G_PARAM_READWRITE);
+
+  g_object_class_install_property (object_class,
+                                   PROP_SHADOW_CLASS,
+                                   pspec);
 }
 
 static void
 meta_window_actor_x11_init (MetaWindowActorX11 *self)
 {
+  /* We do this now since we might be going right back into the frozen state. */
+  g_signal_connect (self, "thawed", G_CALLBACK (handle_updates), NULL);
+
+  self->shadow_factory = meta_shadow_factory_get_default ();
+  self->shadow_factory_changed_handler_id =
+    g_signal_connect (self->shadow_factory,
+                      "changed",
+                      G_CALLBACK (invalidate_shadow),
+                      self);
 }
diff --git a/src/compositor/meta-window-actor-x11.h b/src/compositor/meta-window-actor-x11.h
index 5f20fc028..86b80034d 100644
--- a/src/compositor/meta-window-actor-x11.h
+++ b/src/compositor/meta-window-actor-x11.h
@@ -31,4 +31,17 @@ G_DECLARE_FINAL_TYPE (MetaWindowActorX11,
                       META, WINDOW_ACTOR_X11,
                       MetaWindowActor)
 
+void meta_window_actor_x11_process_x11_damage (MetaWindowActorX11 *actor_x11,
+                                               XDamageNotifyEvent *event);
+
+gboolean meta_window_actor_x11_should_unredirect (MetaWindowActorX11 *actor_x11);
+
+void meta_window_actor_x11_set_unredirected (MetaWindowActorX11 *actor_x11,
+                                             gboolean            unredirected);
+
+void meta_window_actor_x11_update_shape (MetaWindowActorX11 *actor_x11);
+
+void meta_window_actor_x11_process_damage (MetaWindowActorX11 *actor_x11,
+                                           XDamageNotifyEvent *event);
+
 #endif /* META_WINDOW_ACTOR_X11_H */
diff --git a/src/compositor/meta-window-actor.c b/src/compositor/meta-window-actor.c
index fd99844e8..d52ec240a 100644
--- a/src/compositor/meta-window-actor.c
+++ b/src/compositor/meta-window-actor.c
@@ -26,16 +26,14 @@
 #include <string.h>
 
 #include "backends/meta-screen-cast-window.h"
-#include "core/frame.h"
 #include "compositor/compositor-private.h"
 #include "compositor/meta-cullable.h"
 #include "compositor/meta-shaped-texture-private.h"
 #include "compositor/meta-surface-actor-x11.h"
 #include "compositor/meta-surface-actor.h"
 #include "compositor/meta-window-actor-private.h"
-#include "compositor/region-utils.h"
-#include "meta/meta-enum-types.h"
-#include "meta/meta-shadow-factory.h"
+#include "core/boxes-private.h"
+#include "core/window-private.h"
 #include "meta/window.h"
 
 #ifdef HAVE_WAYLAND
@@ -57,35 +55,8 @@ typedef struct _MetaWindowActorPrivate
 
   MetaSurfaceActor *surface;
 
-  /* MetaShadowFactory only caches shadows that are actually in use;
-   * to avoid unnecessary recomputation we do two things: 1) we store
-   * both a focused and unfocused shadow for the window. If the window
-   * doesn't have different focused and unfocused shadow parameters,
-   * these will be the same. 2) when the shadow potentially changes we
-   * don't immediately unreference the old shadow, we just flag it as
-   * dirty and recompute it when we next need it (recompute_focused_shadow,
-   * recompute_unfocused_shadow.) Because of our extraction of
-   * size-invariant window shape, we'll often find that the new shadow
-   * is the same as the old shadow.
-   */
-  MetaShadow       *focused_shadow;
-  MetaShadow       *unfocused_shadow;
-
-  /* A region that matches the shape of the window, including frame bounds */
-  cairo_region_t   *shape_region;
-  /* The region we should clip to when painting the shadow */
-  cairo_region_t   *shadow_clip;
-
-  /* Extracted size-invariant shape used for shadows */
-  MetaWindowShape  *shadow_shape;
-  char *            shadow_class;
-
-  MetaShadowMode    shadow_mode;
-
   int geometry_scale;
 
-  gulong            size_changed_id;
-
   /*
    * These need to be counters rather than flags, since more plugins
    * can implement same effect; the practicality of stacking effects
@@ -102,10 +73,6 @@ typedef struct _MetaWindowActorPrivate
   guint                    visible                : 1;
   guint                    disposed               : 1;
 
-  guint             needs_reshape          : 1;
-  guint             recompute_focused_shadow   : 1;
-  guint             recompute_unfocused_shadow : 1;
-
   guint                    needs_destroy          : 1;
 
   guint             updates_frozen         : 1;
@@ -117,6 +84,7 @@ enum
   FIRST_FRAME,
   EFFECTS_COMPLETED,
   DAMAGED,
+  THAWED,
 
   LAST_SIGNAL
 };
@@ -126,8 +94,6 @@ static guint signals[LAST_SIGNAL] = { 0 };
 enum
 {
   PROP_META_WINDOW = 1,
-  PROP_SHADOW_MODE,
-  PROP_SHADOW_CLASS
 };
 
 static void meta_window_actor_dispose    (GObject *object);
@@ -144,17 +110,6 @@ static void meta_window_actor_get_property (GObject      *object,
 static void meta_window_actor_real_assign_surface_actor (MetaWindowActor  *self,
                                                          MetaSurfaceActor *surface_actor);
 
-static void meta_window_actor_paint (ClutterActor *actor);
-
-static gboolean meta_window_actor_get_paint_volume (ClutterActor       *actor,
-                                                    ClutterPaintVolume *volume);
-
-static gboolean meta_window_actor_has_shadow (MetaWindowActor *self);
-
-static void meta_window_actor_handle_updates (MetaWindowActor *self);
-
-static void check_needs_reshape (MetaWindowActor *self);
-
 static void cullable_iface_init (MetaCullableInterface *iface);
 
 static void screen_cast_window_iface_init (MetaScreenCastWindowInterface *iface);
@@ -168,7 +123,6 @@ static void
 meta_window_actor_class_init (MetaWindowActorClass *klass)
 {
   GObjectClass *object_class = G_OBJECT_CLASS (klass);
-  ClutterActorClass *actor_class = CLUTTER_ACTOR_CLASS (klass);
   GParamSpec   *pspec;
 
   object_class->dispose      = meta_window_actor_dispose;
@@ -176,9 +130,6 @@ meta_window_actor_class_init (MetaWindowActorClass *klass)
   object_class->get_property = meta_window_actor_get_property;
   object_class->constructed  = meta_window_actor_constructed;
 
-  actor_class->paint = meta_window_actor_paint;
-  actor_class->get_paint_volume = meta_window_actor_get_paint_volume;
-
   klass->assign_surface_actor = meta_window_actor_real_assign_surface_actor;
 
   /**
@@ -235,6 +186,18 @@ meta_window_actor_class_init (MetaWindowActorClass *klass)
                   NULL, NULL, NULL,
                   G_TYPE_NONE, 0);
 
+  /**
+   * MetaWindowActor::thawed:
+   * @actor: the #MetaWindowActor instance
+   */
+  signals[THAWED] =
+    g_signal_new ("thawed",
+                  G_TYPE_FROM_CLASS (object_class),
+                  G_SIGNAL_RUN_LAST,
+                  0,
+                  NULL, NULL, NULL,
+                  G_TYPE_NONE, 0);
+
   pspec = g_param_spec_object ("meta-window",
                                "MetaWindow",
                                "The displayed MetaWindow",
@@ -244,27 +207,6 @@ meta_window_actor_class_init (MetaWindowActorClass *klass)
   g_object_class_install_property (object_class,
                                    PROP_META_WINDOW,
                                    pspec);
-
-  pspec = g_param_spec_enum ("shadow-mode",
-                             "Shadow mode",
-                             "Decides when to paint shadows",
-                             META_TYPE_SHADOW_MODE,
-                             META_SHADOW_MODE_AUTO,
-                             G_PARAM_READWRITE);
-
-  g_object_class_install_property (object_class,
-                                   PROP_SHADOW_MODE,
-                                   pspec);
-
-  pspec = g_param_spec_string ("shadow-class",
-                               "Name of the shadow class for this window.",
-                               "NULL means to use the default shadow class for this window type",
-                               NULL,
-                               G_PARAM_READWRITE);
-
-  g_object_class_install_property (object_class,
-                                   PROP_SHADOW_CLASS,
-                                   pspec);
 }
 
 static void
@@ -284,43 +226,24 @@ window_appears_focused_notify (MetaWindow *mw,
   clutter_actor_queue_redraw (CLUTTER_ACTOR (data));
 }
 
-static void
-surface_size_changed (MetaSurfaceActor *actor,
-                      gpointer          user_data)
-{
-  MetaWindowActor *self = META_WINDOW_ACTOR (user_data);
-
-  meta_window_actor_update_shape (self);
-}
-
-static gboolean
-is_argb32 (MetaWindowActor *self)
+gboolean
+meta_window_actor_is_opaque (MetaWindowActor *self)
 {
   MetaWindowActorPrivate *priv =
     meta_window_actor_get_instance_private (self);
+  MetaWindow *window = priv->window;
 
-  /* assume we're argb until we get the window (because
-     in practice we're drawing nothing, so we're fully
-     transparent)
-  */
-  if (priv->surface)
-    return meta_surface_actor_is_argb32 (priv->surface);
-  else
-    return TRUE;
-}
+  if (window->opacity != 0xff)
+    return FALSE;
 
-static gboolean
-is_opaque (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  MetaWindow *window = priv->window;
+  if (!priv->surface)
+    return FALSE;
 
-  return !is_argb32 (self) && (window->opacity == 0xFF);
+  return meta_surface_actor_is_opaque (priv->surface);
 }
 
-static gboolean
-is_frozen (MetaWindowActor *self)
+gboolean
+meta_window_actor_is_frozen (MetaWindowActor *self)
 {
   MetaWindowActorPrivate *priv =
     meta_window_actor_get_instance_private (self);
@@ -370,14 +293,12 @@ meta_window_actor_thaw (MetaWindowActor *self)
     return;
 
   /* We still might be frozen due to lack of a MetaSurfaceActor */
-  if (is_frozen (self))
+  if (meta_window_actor_is_frozen (self))
     return;
 
   meta_window_actor_sync_thawed_state (self);
 
-  /* We do this now since we might be going right back into the
-   * frozen state */
-  meta_window_actor_handle_updates (self);
+  g_signal_emit (self, signals[THAWED], 0);
 }
 
 static void
@@ -387,25 +308,10 @@ meta_window_actor_real_assign_surface_actor (MetaWindowActor  *self,
   MetaWindowActorPrivate *priv =
     meta_window_actor_get_instance_private (self);
 
-  if (priv->surface)
-    {
-      g_warn_if_fail (priv->window->client_type == META_WINDOW_CLIENT_TYPE_X11 &&
-                      meta_is_wayland_compositor ());
-
-      g_clear_signal_handler (&priv->size_changed_id, priv->surface);
-      clutter_actor_remove_child (CLUTTER_ACTOR (self),
-                                  CLUTTER_ACTOR (priv->surface));
-      g_clear_object (&priv->surface);
-    }
-
+  g_clear_object (&priv->surface);
   priv->surface = g_object_ref_sink (surface_actor);
-  priv->size_changed_id = g_signal_connect (priv->surface, "size-changed",
-                                            G_CALLBACK (surface_size_changed),
-                                            self);
-
-  meta_window_actor_update_shape (self);
 
-  if (is_frozen (self))
+  if (meta_window_actor_is_frozen (self))
     meta_surface_actor_set_frozen (priv->surface, TRUE);
   else
     meta_window_actor_sync_thawed_state (self);
@@ -457,13 +363,9 @@ meta_window_actor_constructed (GObject *object)
 
   meta_window_actor_update_opacity (self);
 
-  /* Start off with an empty shape region to maintain the invariant
-   * that it's always set */
-  priv->shape_region = cairo_region_create ();
-
   meta_window_actor_sync_updates_frozen (self);
 
-  if (is_frozen (self))
+  if (meta_window_actor_is_frozen (self))
     priv->first_frame_state = INITIALLY_FROZEN;
   else
     priv->first_frame_state = DRAWING_FIRST_FRAME;
@@ -487,21 +389,12 @@ meta_window_actor_dispose (GObject *object)
 
   priv->disposed = TRUE;
 
-  g_clear_pointer (&priv->shape_region, cairo_region_destroy);
-  g_clear_pointer (&priv->shadow_clip, cairo_region_destroy);
-
-  g_clear_pointer (&priv->shadow_class, g_free);
-  g_clear_pointer (&priv->focused_shadow, meta_shadow_unref);
-  g_clear_pointer (&priv->unfocused_shadow, meta_shadow_unref);
-  g_clear_pointer (&priv->shadow_shape, meta_window_shape_unref);
-
   meta_compositor_remove_window_actor (compositor, self);
 
   g_clear_object (&priv->window);
 
   if (priv->surface)
     {
-      g_clear_signal_handler (&priv->size_changed_id, priv->surface);
       clutter_actor_remove_child (CLUTTER_ACTOR (self),
                                   CLUTTER_ACTOR (priv->surface));
       g_clear_object (&priv->surface);
@@ -527,31 +420,6 @@ meta_window_actor_set_property (GObject      *object,
       g_signal_connect_object (priv->window, "notify::appears-focused",
                                G_CALLBACK (window_appears_focused_notify), self, 0);
       break;
-    case PROP_SHADOW_MODE:
-      {
-        MetaShadowMode newv = g_value_get_enum (value);
-
-        if (newv == priv->shadow_mode)
-          return;
-
-        priv->shadow_mode = newv;
-
-        meta_window_actor_invalidate_shadow (self);
-      }
-      break;
-    case PROP_SHADOW_CLASS:
-      {
-        const char *newv = g_value_get_string (value);
-
-        if (g_strcmp0 (newv, priv->shadow_class) == 0)
-          return;
-
-        g_free (priv->shadow_class);
-        priv->shadow_class = g_strdup (newv);
-
-        meta_window_actor_invalidate_shadow (self);
-      }
-      break;
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
@@ -573,270 +441,12 @@ meta_window_actor_get_property (GObject      *object,
     case PROP_META_WINDOW:
       g_value_set_object (value, priv->window);
       break;
-    case PROP_SHADOW_MODE:
-      g_value_set_enum (value, priv->shadow_mode);
-      break;
-    case PROP_SHADOW_CLASS:
-      g_value_set_string (value, priv->shadow_class);
-      break;
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
     }
 }
 
-static const char *
-meta_window_actor_get_shadow_class (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  if (priv->shadow_class != NULL)
-    return priv->shadow_class;
-  else
-    {
-      MetaWindowType window_type = meta_window_get_window_type (priv->window);
-
-      switch (window_type)
-        {
-        case META_WINDOW_DROPDOWN_MENU:
-        case META_WINDOW_COMBO:
-          return "dropdown-menu";
-        case META_WINDOW_POPUP_MENU:
-          return "popup-menu";
-        default:
-          {
-            MetaFrameType frame_type = meta_window_get_frame_type (priv->window);
-            return meta_frame_type_to_string (frame_type);
-          }
-        }
-    }
-}
-
-static void
-meta_window_actor_get_shadow_params (MetaWindowActor  *self,
-                                     gboolean          appears_focused,
-                                     MetaShadowParams *params)
-{
-  const char *shadow_class = meta_window_actor_get_shadow_class (self);
-
-  meta_shadow_factory_get_params (meta_shadow_factory_get_default (),
-                                  shadow_class, appears_focused,
-                                  params);
-}
-
-void
-meta_window_actor_get_shape_bounds (MetaWindowActor       *self,
-                                    cairo_rectangle_int_t *bounds)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  cairo_region_get_extents (priv->shape_region, bounds);
-}
-
-static void
-meta_window_actor_get_shadow_bounds (MetaWindowActor       *self,
-                                     gboolean               appears_focused,
-                                     cairo_rectangle_int_t *bounds)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  MetaShadow *shadow;
-  cairo_rectangle_int_t shape_bounds;
-  MetaShadowParams params;
-
-  shadow = appears_focused ? priv->focused_shadow : priv->unfocused_shadow;
-
-  meta_window_actor_get_shape_bounds (self, &shape_bounds);
-  meta_window_actor_get_shadow_params (self, appears_focused, &params);
-
-  meta_shadow_get_bounds (shadow,
-                          params.x_offset + shape_bounds.x,
-                          params.y_offset + shape_bounds.y,
-                          shape_bounds.width,
-                          shape_bounds.height,
-                          bounds);
-}
-
-/* If we have an ARGB32 window that we decorate with a frame, it's
- * probably something like a translucent terminal - something where
- * the alpha channel represents transparency rather than a shape.  We
- * don't want to show the shadow through the translucent areas since
- * the shadow is wrong for translucent windows (it should be
- * translucent itself and colored), and not only that, will /look/
- * horribly wrong - a misplaced big black blob. As a hack, what we
- * want to do is just draw the shadow as normal outside the frame, and
- * inside the frame draw no shadow.  This is also not even close to
- * the right result, but looks OK. We also apply this approach to
- * windows set to be partially translucent with _NET_WM_WINDOW_OPACITY.
- */
-static gboolean
-clip_shadow_under_window (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  if (priv->window->frame)
-    return TRUE;
-
-  return is_opaque (self);
-}
-
-static void
-meta_window_actor_paint (ClutterActor *actor)
-{
-  MetaWindowActor *self = META_WINDOW_ACTOR (actor);
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  gboolean appears_focused = meta_window_appears_focused (priv->window);
-  MetaShadow *shadow;
-  CoglFramebuffer *framebuffer = cogl_get_draw_framebuffer ();
-
-  shadow = appears_focused ? priv->focused_shadow : priv->unfocused_shadow;
-
-  if (shadow != NULL)
-    {
-      MetaShadowParams params;
-      cairo_rectangle_int_t shape_bounds;
-      cairo_region_t *clip = priv->shadow_clip;
-      MetaWindow *window = priv->window;
-
-      meta_window_actor_get_shape_bounds (self, &shape_bounds);
-      meta_window_actor_get_shadow_params (self, appears_focused, &params);
-
-      /* The frame bounds are already subtracted from priv->shadow_clip
-       * if that exists.
-       */
-      if (!clip && clip_shadow_under_window (self))
-        {
-          cairo_region_t *frame_bounds = meta_window_get_frame_bounds (priv->window);
-          cairo_rectangle_int_t bounds;
-
-          meta_window_actor_get_shadow_bounds (self, appears_focused, &bounds);
-          clip = cairo_region_create_rectangle (&bounds);
-
-          if (frame_bounds)
-            cairo_region_subtract (clip, frame_bounds);
-        }
-
-      meta_shadow_paint (shadow,
-                         framebuffer,
-                         params.x_offset + shape_bounds.x,
-                         params.y_offset + shape_bounds.y,
-                         shape_bounds.width,
-                         shape_bounds.height,
-                         (clutter_actor_get_paint_opacity (actor) * params.opacity * window->opacity) / (255 
* 255),
-                         clip,
-                         clip_shadow_under_window (self)); /* clip_strictly - not just as an optimization */
-
-      if (clip && clip != priv->shadow_clip)
-        cairo_region_destroy (clip);
-    }
-
-  CLUTTER_ACTOR_CLASS (meta_window_actor_parent_class)->paint (actor);
-}
-
-static gboolean
-meta_window_actor_get_paint_volume (ClutterActor       *actor,
-                                    ClutterPaintVolume *volume)
-{
-  MetaWindowActor *self = META_WINDOW_ACTOR (actor);
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  gboolean appears_focused = meta_window_appears_focused (priv->window);
-
-  /* The paint volume is computed before paint functions are called
-   * so our bounds might not be updated yet. Force an update. */
-  meta_window_actor_handle_updates (self);
-
-  if (appears_focused ? priv->focused_shadow : priv->unfocused_shadow)
-    {
-      cairo_rectangle_int_t shadow_bounds;
-      ClutterActorBox shadow_box;
-
-      /* We could compute an full clip region as we do for the window
-       * texture, but the shadow is relatively cheap to draw, and
-       * a little more complex to clip, so we just catch the case where
-       * the shadow is completely obscured and doesn't need to be drawn
-       * at all.
-       */
-
-      meta_window_actor_get_shadow_bounds (self, appears_focused, &shadow_bounds);
-      shadow_box.x1 = shadow_bounds.x;
-      shadow_box.x2 = shadow_bounds.x + shadow_bounds.width;
-      shadow_box.y1 = shadow_bounds.y;
-      shadow_box.y2 = shadow_bounds.y + shadow_bounds.height;
-
-      clutter_paint_volume_union_box (volume, &shadow_box);
-    }
-
-  if (priv->surface)
-    {
-      const ClutterPaintVolume *child_volume;
-
-      child_volume = clutter_actor_get_transformed_paint_volume (CLUTTER_ACTOR (priv->surface), actor);
-      if (!child_volume)
-        return FALSE;
-
-      clutter_paint_volume_union (volume, child_volume);
-    }
-
-  return TRUE;
-}
-
-static gboolean
-meta_window_actor_has_shadow (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  if (priv->shadow_mode == META_SHADOW_MODE_FORCED_OFF)
-    return FALSE;
-  if (priv->shadow_mode == META_SHADOW_MODE_FORCED_ON)
-    return TRUE;
-
-  /* Leaving out shadows for maximized and fullscreen windows is an effeciency
-   * win and also prevents the unsightly effect of the shadow of maximized
-   * window appearing on an adjacent window */
-  if ((meta_window_get_maximized (priv->window) == META_MAXIMIZE_BOTH) ||
-      meta_window_is_fullscreen (priv->window))
-    return FALSE;
-
-  /*
-   * If we have two snap-tiled windows, we don't want the shadow to obstruct
-   * the other window.
-   */
-  if (meta_window_get_tile_match (priv->window))
-    return FALSE;
-
-  /*
-   * Always put a shadow around windows with a frame - This should override
-   * the restriction about not putting a shadow around ARGB windows.
-   */
-  if (meta_window_get_frame (priv->window))
-    return TRUE;
-
-  /*
-   * Do not add shadows to non-opaque (ARGB32) windows, as we can't easily
-   * generate shadows for them.
-   */
-  if (!is_opaque (self))
-    return FALSE;
-
-  /*
-   * If a window specifies that it has custom frame extents, that likely
-   * means that it is drawing a shadow itself. Don't draw our own.
-   */
-  if (priv->window->has_custom_frame_extents)
-    return FALSE;
-
-  /*
-   * Generate shadows for all other windows.
-   */
-  return TRUE;
-}
-
 /**
  * meta_window_actor_get_meta_window:
  * @self: a #MetaWindowActor
@@ -1100,29 +710,6 @@ meta_window_actor_effect_completed (MetaWindowActor  *self,
     meta_window_actor_after_effects (self);
 }
 
-gboolean
-meta_window_actor_should_unredirect (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  if (!meta_window_actor_is_destroyed (self) && priv->surface)
-    return meta_surface_actor_should_unredirect (priv->surface);
-  else
-    return FALSE;
-}
-
-void
-meta_window_actor_set_unredirected (MetaWindowActor *self,
-                                    gboolean         unredirected)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  g_assert (priv->surface); /* because otherwise should_unredirect() is FALSE */
-  meta_surface_actor_set_unredirected (priv->surface, unredirected);
-}
-
 void
 meta_window_actor_queue_destroy (MetaWindowActor *self)
 {
@@ -1181,7 +768,7 @@ meta_window_actor_sync_actor_geometry (MetaWindowActor *self,
    * is shown, the map effect will go into effect and prevent further geometry
    * updates.
    */
-  if (is_frozen (self) && !did_placement)
+  if (meta_window_actor_is_frozen (self) && !did_placement)
     return META_WINDOW_ACTOR_CHANGE_POSITION | META_WINDOW_ACTOR_CHANGE_SIZE;
 
   if (meta_window_actor_effect_in_progress (self))
@@ -1358,45 +945,6 @@ see_region (cairo_region_t *region,
 }
 #endif
 
-/**
- * meta_window_actor_set_clip_region_beneath:
- * @self: a #MetaWindowActor
- * @clip_region: the region of the screen that isn't completely
- *  obscured beneath the main window texture.
- *
- * Provides a hint as to what areas need to be drawn *beneath*
- * the main window texture.  This is the relevant clip region
- * when drawing the shadow, properly accounting for areas of the
- * shadow hid by the window itself. This will be set before painting
- * then unset afterwards.
- */
-static void
-meta_window_actor_set_clip_region_beneath (MetaWindowActor *self,
-                                           cairo_region_t  *beneath_region)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  gboolean appears_focused = meta_window_appears_focused (priv->window);
-
-  if (appears_focused ? priv->focused_shadow : priv->unfocused_shadow)
-    {
-      g_clear_pointer (&priv->shadow_clip, cairo_region_destroy);
-
-      if (beneath_region)
-        {
-          priv->shadow_clip = cairo_region_copy (beneath_region);
-
-          if (clip_shadow_under_window (self))
-            {
-              cairo_region_t *frame_bounds = meta_window_get_frame_bounds (priv->window);
-              if (frame_bounds)
-                cairo_region_subtract (priv->shadow_clip, frame_bounds);
-            }
-        }
-      else
-        priv->shadow_clip = NULL;
-    }
-}
 
 static void
 meta_window_actor_cull_out (MetaCullable   *cullable,
@@ -1408,9 +956,8 @@ meta_window_actor_cull_out (MetaCullable   *cullable,
     meta_window_actor_get_instance_private (self);
 
   meta_cullable_cull_out_children (cullable, unobscured_region, clip_region);
-  meta_window_actor_set_clip_region_beneath (self, clip_region);
 
-  if (unobscured_region && is_opaque (self))
+  if (unobscured_region && meta_window_actor_is_opaque (self))
     {
       cairo_region_t *region = meta_window_get_frame_bounds (priv->window);
 
@@ -1431,12 +978,6 @@ meta_window_actor_cull_out (MetaCullable   *cullable,
 static void
 meta_window_actor_reset_culling (MetaCullable *cullable)
 {
-  MetaWindowActor *self = META_WINDOW_ACTOR (cullable);
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  g_clear_pointer (&priv->shadow_clip, cairo_region_destroy);
-
   meta_cullable_reset_culling_children (cullable);
 }
 
@@ -1447,86 +988,6 @@ cullable_iface_init (MetaCullableInterface *iface)
   iface->reset_culling = meta_window_actor_reset_culling;
 }
 
-static void
-check_needs_shadow (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  MetaShadow *old_shadow = NULL;
-  MetaShadow **shadow_location;
-  gboolean recompute_shadow;
-  gboolean should_have_shadow;
-  gboolean appears_focused;
-
-  /* Calling meta_window_actor_has_shadow() here at every pre-paint is cheap
-   * and avoids the need to explicitly handle window type changes, which
-   * we would do if tried to keep track of when we might be adding or removing
-   * a shadow more explicitly. We only keep track of changes to the *shape* of
-   * the shadow with priv->recompute_shadow.
-   */
-
-  should_have_shadow = meta_window_actor_has_shadow (self);
-  appears_focused = meta_window_appears_focused (priv->window);
-
-  if (appears_focused)
-    {
-      recompute_shadow = priv->recompute_focused_shadow;
-      priv->recompute_focused_shadow = FALSE;
-      shadow_location = &priv->focused_shadow;
-    }
-  else
-    {
-      recompute_shadow = priv->recompute_unfocused_shadow;
-      priv->recompute_unfocused_shadow = FALSE;
-      shadow_location = &priv->unfocused_shadow;
-    }
-
-  if (!should_have_shadow || recompute_shadow)
-    {
-      if (*shadow_location != NULL)
-        {
-          old_shadow = *shadow_location;
-          *shadow_location = NULL;
-        }
-    }
-
-  if (*shadow_location == NULL && should_have_shadow)
-    {
-      if (priv->shadow_shape == NULL)
-        priv->shadow_shape = meta_window_shape_new (priv->shape_region);
-
-      MetaShadowFactory *factory = meta_shadow_factory_get_default ();
-      const char *shadow_class = meta_window_actor_get_shadow_class (self);
-      cairo_rectangle_int_t shape_bounds;
-
-      meta_window_actor_get_shape_bounds (self, &shape_bounds);
-      *shadow_location = meta_shadow_factory_get_shadow (factory,
-                                                         priv->shadow_shape,
-                                                         shape_bounds.width, shape_bounds.height,
-                                                         shadow_class, appears_focused);
-    }
-
-  if (old_shadow != NULL)
-    meta_shadow_unref (old_shadow);
-}
-
-void
-meta_window_actor_process_x11_damage (MetaWindowActor    *self,
-                                      XDamageNotifyEvent *event)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  if (priv->surface)
-    meta_surface_actor_process_damage (priv->surface,
-                                       event->area.x,
-                                       event->area.y,
-                                       event->area.width,
-                                       event->area.height);
-
-  meta_window_actor_notify_damaged (self);
-}
-
 void
 meta_window_actor_sync_visibility (MetaWindowActor *self)
 {
@@ -1542,295 +1003,12 @@ meta_window_actor_sync_visibility (MetaWindowActor *self)
     }
 }
 
-static cairo_region_t *
-scan_visible_region (guchar         *mask_data,
-                     int             stride,
-                     cairo_region_t *scan_area)
-{
-  int i, n_rects = cairo_region_num_rectangles (scan_area);
-  MetaRegionBuilder builder;
-
-  meta_region_builder_init (&builder);
-
-  for (i = 0; i < n_rects; i++)
-    {
-      int x, y;
-      cairo_rectangle_int_t rect;
-
-      cairo_region_get_rectangle (scan_area, i, &rect);
-
-      for (y = rect.y; y < (rect.y + rect.height); y++)
-        {
-          for (x = rect.x; x < (rect.x + rect.width); x++)
-            {
-              int x2 = x;
-              while (mask_data[y * stride + x2] == 255 && x2 < (rect.x + rect.width))
-                x2++;
-
-              if (x2 > x)
-                {
-                  meta_region_builder_add_rectangle (&builder, x, y, x2 - x, 1);
-                  x = x2;
-                }
-            }
-        }
-    }
-
-  return meta_region_builder_finish (&builder);
-}
-
-static void
-build_and_scan_frame_mask (MetaWindowActor       *self,
-                           cairo_rectangle_int_t *client_area,
-                           cairo_region_t        *shape_region)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  ClutterBackend *backend = clutter_get_default_backend ();
-  CoglContext *ctx = clutter_backend_get_cogl_context (backend);
-  guchar *mask_data;
-  guint tex_width, tex_height;
-  MetaShapedTexture *stex;
-  CoglTexture *paint_tex, *mask_texture;
-  int stride;
-  cairo_t *cr;
-  cairo_surface_t *surface;
-  GError *error = NULL;
-
-  stex = meta_surface_actor_get_texture (priv->surface);
-  g_return_if_fail (stex);
-
-  meta_shaped_texture_set_mask_texture (stex, NULL);
-
-  paint_tex = meta_shaped_texture_get_texture (stex);
-  if (paint_tex == NULL)
-    return;
-
-  tex_width = cogl_texture_get_width (paint_tex);
-  tex_height = cogl_texture_get_height (paint_tex);
-
-  stride = cairo_format_stride_for_width (CAIRO_FORMAT_A8, tex_width);
-
-  /* Create data for an empty image */
-  mask_data = g_malloc0 (stride * tex_height);
-
-  surface = cairo_image_surface_create_for_data (mask_data,
-                                                 CAIRO_FORMAT_A8,
-                                                 tex_width,
-                                                 tex_height,
-                                                 stride);
-  cr = cairo_create (surface);
-
-  gdk_cairo_region (cr, shape_region);
-  cairo_fill (cr);
-
-  if (priv->window->frame != NULL)
-    {
-      cairo_region_t *frame_paint_region, *scanned_region;
-      cairo_rectangle_int_t rect = { 0, 0, tex_width, tex_height };
-
-      /* Make sure we don't paint the frame over the client window. */
-      frame_paint_region = cairo_region_create_rectangle (&rect);
-      cairo_region_subtract_rectangle (frame_paint_region, client_area);
-
-      gdk_cairo_region (cr, frame_paint_region);
-      cairo_clip (cr);
-
-      meta_frame_get_mask (priv->window->frame, cr);
-
-      cairo_surface_flush (surface);
-      scanned_region = scan_visible_region (mask_data, stride, frame_paint_region);
-      cairo_region_union (shape_region, scanned_region);
-      cairo_region_destroy (scanned_region);
-      cairo_region_destroy (frame_paint_region);
-    }
-
-  cairo_destroy (cr);
-  cairo_surface_destroy (surface);
-
-  mask_texture = COGL_TEXTURE (cogl_texture_2d_new_from_data (ctx, tex_width, tex_height,
-                                                              COGL_PIXEL_FORMAT_A_8,
-                                                              stride, mask_data, &error));
-
-  if (error)
-    {
-      g_warning ("Failed to allocate mask texture: %s", error->message);
-      g_error_free (error);
-    }
-
-  meta_shaped_texture_set_mask_texture (stex, mask_texture);
-  if (mask_texture)
-    cogl_object_unref (mask_texture);
-
-  g_free (mask_data);
-}
-
-static void
-meta_window_actor_update_shape_region (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  cairo_region_t *region = NULL;
-  cairo_rectangle_int_t client_area;
-
-  meta_window_get_client_area_rect (priv->window, &client_area);
-
-  if (priv->window->frame != NULL && priv->window->shape_region != NULL)
-    {
-      region = cairo_region_copy (priv->window->shape_region);
-      cairo_region_translate (region, client_area.x, client_area.y);
-    }
-  else if (priv->window->shape_region != NULL)
-    {
-      region = cairo_region_reference (priv->window->shape_region);
-    }
-  else
-    {
-      /* If we don't have a shape on the server, that means that
-       * we have an implicit shape of one rectangle covering the
-       * entire window. */
-      region = cairo_region_create_rectangle (&client_area);
-    }
-
-  if ((priv->window->shape_region != NULL) || (priv->window->frame != NULL))
-    build_and_scan_frame_mask (self, &client_area, region);
-
-  g_clear_pointer (&priv->shape_region, cairo_region_destroy);
-  priv->shape_region = region;
-
-  g_clear_pointer (&priv->shadow_shape, meta_window_shape_unref);
-
-  meta_window_actor_invalidate_shadow (self);
-}
-
-static void
-meta_window_actor_update_input_region (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  MetaWindow *window = priv->window;
-  cairo_region_t *region;
-
-  if (window->shape_region && window->input_region)
-    {
-      region = cairo_region_copy (window->shape_region);
-      cairo_region_intersect (region, window->input_region);
-    }
-  else if (window->shape_region)
-    region = cairo_region_reference (window->shape_region);
-  else if (window->input_region)
-    region = cairo_region_reference (window->input_region);
-  else
-    region = NULL;
-
-  meta_surface_actor_set_input_region (priv->surface, region);
-  cairo_region_destroy (region);
-}
-
-static void
-meta_window_actor_update_opaque_region (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-  cairo_region_t *opaque_region;
-  gboolean argb32 = is_argb32 (self);
-
-  if (argb32 && priv->window->opaque_region != NULL)
-    {
-      cairo_rectangle_int_t client_area;
-
-      meta_window_get_client_area_rect (priv->window, &client_area);
-
-      /* The opaque region is defined to be a part of the
-       * window which ARGB32 will always paint with opaque
-       * pixels. For these regions, we want to avoid painting
-       * windows and shadows beneath them.
-       *
-       * If the client gives bad coordinates where it does not
-       * fully paint, the behavior is defined by the specification
-       * to be undefined, and considered a client bug. In mutter's
-       * case, graphical glitches will occur.
-       */
-      opaque_region = cairo_region_copy (priv->window->opaque_region);
-      cairo_region_translate (opaque_region, client_area.x, client_area.y);
-      cairo_region_intersect (opaque_region, priv->shape_region);
-    }
-  else if (argb32)
-    opaque_region = NULL;
-  else
-    opaque_region = cairo_region_reference (priv->shape_region);
-
-  meta_surface_actor_set_opaque_region (priv->surface, opaque_region);
-  cairo_region_destroy (opaque_region);
-}
-
-static void
-check_needs_reshape (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  if (!priv->needs_reshape)
-    return;
-
-  meta_window_actor_update_shape_region (self);
-
-  if (priv->window->client_type == META_WINDOW_CLIENT_TYPE_X11)
-    {
-      meta_window_actor_update_input_region (self);
-      meta_window_actor_update_opaque_region (self);
-    }
-
-  priv->needs_reshape = FALSE;
-}
-
-void
-meta_window_actor_update_shape (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  priv->needs_reshape = TRUE;
-
-  if (is_frozen (self))
-    return;
-
-  clutter_actor_queue_redraw (CLUTTER_ACTOR (priv->surface));
-}
-
-static void
-meta_window_actor_handle_updates (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  if (is_frozen (self))
-    {
-      /* The window is frozen due to a pending animation: we'll wait until
-       * the animation finishes to reshape and repair the window */
-      return;
-    }
-
-  if (meta_surface_actor_is_unredirected (priv->surface))
-    return;
-
-  meta_surface_actor_pre_paint (priv->surface);
-
-  if (!meta_surface_actor_is_visible (priv->surface))
-    return;
-
-  check_needs_reshape (self);
-  check_needs_shadow (self);
-}
-
 void
 meta_window_actor_pre_paint (MetaWindowActor *self)
 {
   if (meta_window_actor_is_destroyed (self))
     return;
 
-  meta_window_actor_handle_updates (self);
-
   META_WINDOW_ACTOR_GET_CLASS (self)->pre_paint (self);
 }
 
@@ -1862,21 +1040,6 @@ meta_window_actor_frame_complete (MetaWindowActor  *self,
                                                       presentation_time);
 }
 
-void
-meta_window_actor_invalidate_shadow (MetaWindowActor *self)
-{
-  MetaWindowActorPrivate *priv =
-    meta_window_actor_get_instance_private (self);
-
-  priv->recompute_focused_shadow = TRUE;
-  priv->recompute_unfocused_shadow = TRUE;
-
-  if (is_frozen (self))
-    return;
-
-  clutter_actor_queue_redraw (CLUTTER_ACTOR (self));
-}
-
 void
 meta_window_actor_update_opacity (MetaWindowActor *self)
 {


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