e_compositor: Remove obsolte implementation sandbox/shiin/devel
authorSeunghun Lee <shiin.lee@samsung.com>
Thu, 22 Aug 2024 09:43:38 +0000 (18:43 +0900)
committerSeunghun Lee <shiin.lee@samsung.com>
Thu, 29 Aug 2024 06:50:58 +0000 (15:50 +0900)
This Removes redundant emission of
E_COMP_WL_HOOK_SUBSURFACE_SYNCHRONIZED_COMMIT hook point since it's been
emitted when precommit_from_cache handler is called.

src/bin/server/e_compositor.c

index 2a54d7281b1e0450901843d082c36694474cc32c..7d7676eab8182c5085900dfa33489ece52d2b6d7 100644 (file)
@@ -121,7 +121,6 @@ static E_Subsurface *_e_subsurface_try_from_surface(E_Surface *surface);
 static E_Subsurface *_e_subsurface_try_from_resource(struct wl_resource *resource);
 static E_Subsurface *_e_subsurface_try_from_ds_subsurface(struct ds_subsurface *ds_subsurface);
 static void _e_subsurface_destroy(E_Subsurface *sub);
-static void _e_subsurface_commit(E_Subsurface *sub);
 static void _e_subsurface_place_below_parent(E_Subsurface *sub);
 static Eina_Bool _e_subsurface_synchronized_check(E_Subsurface *sub);
 static void _e_subsurface_cb_destroy(struct wl_listener *listener, void *data);
@@ -270,18 +269,17 @@ EINTERN Eina_Bool
 e_comp_wl_subsurface_commit(E_Client *ec)
 {
    E_Surface *surface;
-   E_Subsurface *sub;
 
    EINA_SAFETY_ON_NULL_RETURN_VAL(ec, EINA_FALSE);
 
-   if (e_object_is_del(E_OBJECT(ec)) || !ec->comp_data) return EINA_FALSE;
+   surface = e_surface_try_from_ec(ec);
+   if (!surface)
+     return EINA_FALSE;
 
-   surface = e_surface_from_ec(ec);
-   sub = _e_subsurface_try_from_surface(surface);
-   if (!sub)
+   if (!_e_subsurface_try_from_surface(surface))
      return EINA_FALSE;
 
-   _e_subsurface_commit(sub);
+   e_comp_wl_surface_commit(ec);
 
    return EINA_TRUE;
 }
@@ -1549,18 +1547,6 @@ _e_subsurface_synchronized_check(E_Subsurface *sub)
    return EINA_FALSE;
 }
 
-static void
-_e_subsurface_commit(E_Subsurface *sub)
-{
-   e_comp_wl_surface_commit(sub->surface->ec);
-
-   if (_e_subsurface_synchronized_check(sub))
-     {
-        e_comp_wl_hook_call(E_COMP_WL_HOOK_SUBSURFACE_SYNCHRONIZED_COMMIT,
-                            sub->surface->ec);
-     }
-}
-
 static void
 _e_subsurface_place_below_parent(E_Subsurface *sub)
 {