GestureTarget: Require actor only for RootTarget

Author: leolost2605

lib/Gestures/ActorTarget.vala

@@ -11,12 +11,6 @@
  * If a new child (or target via {@link add_target}) is added, its progress will be synced.
  */
 public class Gala.ActorTarget : Clutter.Actor, GestureTarget {
-    public Clutter.Actor? actor {
-        get {
-            return this;
-        }
-    }
-
     public bool animating { get { return ongoing_animations > 0; } }
 
     private double[] current_progress;

lib/Gestures/GestureTarget.vala

@@ -13,11 +13,5 @@ public interface Gala.GestureTarget : Object {
         END
     }
 
-    /**
-     * The actor manipulated by the gesture. The associated frame clock
-     * will be used for animation timelines.
-     */
-    public abstract Clutter.Actor? actor { get; }
-
     public virtual void propagate (UpdateType update_type, GestureAction action, double progress) { }
 }

lib/Gestures/RootTarget.vala

@@ -6,6 +6,12 @@
  */
 
 public interface Gala.RootTarget : Object, GestureTarget {
+    /**
+     * The actor manipulated by the gesture. The associated frame clock
+     * will be used for animation timelines.
+     */
+    public abstract Clutter.Actor? actor { get; }
+
     public void add_gesture_controller (GestureController controller) requires (controller.target == null) {
         controller.attached (this);
         weak_ref (controller.detached);

src/ShellClients/ShellClientsManager.vala

@@ -20,8 +20,6 @@ public class Gala.ShellClientsManager : Object, GestureTarget {
         return instance;
     }
 
-    public Clutter.Actor? actor { get { return wm.stage; } }
-
     public WindowManager wm { get; construct; }
 
     private NotificationsClient notifications_client;

src/ShellClients/ShellWindow.vala

@@ -6,7 +6,6 @@
  */
 
 public class Gala.ShellWindow : PositionedWindow, GestureTarget {
-    public Clutter.Actor? actor { get { return window_actor; } }
     public bool restore_previous_x11_region { private get; set; default = false; }
     public bool visible_in_multitasking_view { get; set; default = false; }
 

src/Widgets/MultitaskingView/MultitaskingView.vala

@@ -28,6 +28,8 @@ public class Gala.MultitaskingView : ActorTarget, RootTarget, ActivatableCompone
 
     public WindowManagerGala wm { get; construct; }
 
+    public Clutter.Actor? actor { get { return this; } }
+
     private Meta.Display display;
     private ModalProxy modal_proxy;
     private bool opened = false;

src/Widgets/MultitaskingView/WindowClone.vala

@@ -30,6 +30,8 @@ public class Gala.WindowClone : ActorTarget, RootTarget {
     public WindowManager wm { get; construct; }
     public Meta.Window window { get; construct; }
 
+    public Clutter.Actor? actor { get { return this; } }
+
     /**
      * The currently assigned slot of the window in the tiling layout. May be null.
      */

src/Widgets/WindowOverview.vala

@@ -12,6 +12,8 @@ public class Gala.WindowOverview : ActorTarget, RootTarget, ActivatableComponent
 
     public WindowManager wm { get; construct; }
 
+    public Clutter.Actor? actor { get { return this; } }
+
     private GestureController gesture_controller; // Currently not used for actual touchpad gestures but only as controller
 
     private ModalProxy modal_proxy;