Revert "handle multiple input regions for csd clients"

This reverts commit 950bee7ce406ced7ab98a3ee6b964fed813b6ce1.

accidentally pushed wrong ref, multiple incomplete broken commits were
merged

:(

fix T5739

3 files changed, 61 insertions, 84 deletions

diff --git a/src/bin/e_comp_object.c b/src/bin/e_comp_object.c
index ae97a83a98..cf53c8020b 100644
--- a/src/bin/e_comp_object.c
+++ b/src/bin/e_comp_object.c
@@ -56,7 +56,7 @@ typedef struct _E_Comp_Object
    //EINA_INLIST;
 
    int                  x, y, w, h;  // geometry
-   Eina_Tiler          *input_area;
+   Eina_Rectangle       input_rect;
 
    E_Client *ec;
 
@@ -76,7 +76,7 @@ typedef struct _E_Comp_Object
 
    Evas_Object         *smart_obj;  // smart object
    Evas_Object         *clip; // clipper over effect object
-   Eina_Array          *input_objs; // input rect
+   Evas_Object         *input_obj; // input rect
    Evas_Object         *obj;  // composite object
    Evas_Object         *frame_object; // for client frames
    Evas_Object         *frame_icon; // for client frames
@@ -765,7 +765,7 @@ _e_comp_object_shadow_setup(E_Comp_Object *cw)
         else
           edje_object_part_swallow(cw->shobj, "e.swallow.content", cw->obj);
      }
-   if (cw->input_objs)
+   if (cw->input_obj)
      evas_object_pass_events_set(cw->obj, 1);
    else
      evas_object_pass_events_set(cw->obj, 0);
@@ -1937,7 +1937,7 @@ _e_comp_object_frame_recalc(E_Comp_Object *cw)
         cw->client_inset.b = MAX(h, 50) - (oy + oh);
         if (cw->obj) edje_object_part_swallow(cw->frame_object, "e.swallow.client", cw->obj);
         evas_object_resize(cw->frame_object, w, h);
-        if (!cw->input_objs)
+        if (cw->input_rect.w && (!cw->input_obj))
           evas_object_pass_events_set(cw->obj, 0);
      }
    else
@@ -2361,6 +2361,7 @@ _e_comp_smart_hide(Evas_Object *obj)
    cw->visible = 0;
    cw->deleted |= cw->ec->delete_requested || e_object_is_del(E_OBJECT(cw->ec));
    evas_object_hide(cw->clip);
+   if (cw->input_obj) evas_object_hide(cw->input_obj);
    evas_object_hide(cw->effect_obj);
    if (stopping) return;
    if (!cw->ec->input_only)
@@ -2396,6 +2397,7 @@ _e_comp_smart_show(Evas_Object *obj)
      evas_object_show(tmp->frame);
 
    evas_object_show(cw->clip);
+   if (cw->input_obj) evas_object_show(cw->input_obj);
    if (!cw->ec->input_only)
      {
         edje_object_thaw(cw->effect_obj);
@@ -2442,19 +2444,6 @@ _e_comp_object_client_del(void *d, void *obj EINA_UNUSED)
 }
 
 static void
-_e_comp_input_objs_free(E_Comp_Object *cw)
-{
-   Evas_Object *rect;
-   if (!cw->input_objs) return;
-   while ((rect = eina_array_pop(cw->input_objs)))
-     {
-        evas_object_smart_member_del(rect);
-        evas_object_freeze_events_set(rect, 1);
-        evas_object_del(rect);
-     }
-}
-
-static void
 _e_comp_smart_del(Evas_Object *obj)
 {
    Eina_List *l;
@@ -2492,9 +2481,7 @@ _e_comp_smart_del(Evas_Object *obj)
    evas_object_del(cw->frame_volume);
    evas_object_del(cw->frame_object);
    evas_object_del(cw->zoomobj);
-   _e_comp_input_objs_free(cw);
-   eina_array_free(cw->input_objs);
-   eina_tiler_free(cw->input_area);
+   evas_object_del(cw->input_obj);
    evas_object_del(cw->obj);
    e_comp_shape_queue();
    eina_stringshare_del(cw->frame_theme);
@@ -2515,25 +2502,15 @@ static void
 _e_comp_object_input_rect_update(E_Comp_Object *cw)
 {
    int x, y, w, h;
-   Eina_Iterator *it;
-   Eina_Rectangle *rect;
-   Evas_Object *o;
-   int i = 0;
 
-   if (!cw->input_area) return;
-   it = eina_tiler_iterator_new(cw->input_area);
-   EINA_ITERATOR_FOREACH(it, rect)
-     {
-        o = eina_array_data_get(cw->input_objs, i++);
-        x = rect->x, y = rect->y, w = rect->w, h = rect->h;
+   if (!cw->input_obj) return;
+   x = cw->input_rect.x, y = cw->input_rect.y, w = cw->input_rect.w, h = cw->input_rect.h;
 
-        E_RECTS_CLIP_TO_RECT(x, y, w, h, 0, 0, cw->ec->client.w, cw->ec->client.h);
-        evas_object_geometry_set(o,
-            cw->x + x + (!!cw->frame_object * cw->client_inset.l),
-            cw->y + y + (!!cw->frame_object * cw->client_inset.t),
-            w, h);
-     }
-   eina_iterator_free(it);
+   E_RECTS_CLIP_TO_RECT(x, y, w, h, 0, 0, cw->ec->client.w, cw->ec->client.h);
+   evas_object_geometry_set(cw->input_obj,
+       cw->x + x + (!!cw->frame_object * cw->client_inset.l),
+       cw->y + y + (!!cw->frame_object * cw->client_inset.t),
+       w, h);
 }
 
 static void
@@ -2551,7 +2528,7 @@ _e_comp_smart_move(Evas_Object *obj, int x, int y)
    cw->agent_updating = 0;
    evas_object_move(cw->clip, 0, 0);
    evas_object_move(cw->effect_obj, x, y);
-   if (cw->input_area)
+   if (cw->input_obj)
      _e_comp_object_input_rect_update(cw);
    /* this gets called once during setup to init coords offscreen and guarantee first move */
    if (e_comp && cw->visible)
@@ -2596,7 +2573,7 @@ _e_comp_smart_resize(Evas_Object *obj, int w, int h)
           }
         evas_object_resize(cw->effect_obj, w, h);
         if (cw->zoomobj) e_zoomap_child_resize(cw->zoomobj, pw, ph);
-        if (cw->input_area)
+        if (cw->input_obj)
           _e_comp_object_input_rect_update(cw);
         /* resize render update tiler */
         if (!first)
@@ -3235,49 +3212,36 @@ e_comp_object_util_center_pos_get(Evas_Object *obj, int *x, int *y)
 }
 
 E_API void
-e_comp_object_input_area_set(Evas_Object *obj, const Eina_Tiler *area)
+e_comp_object_input_area_set(Evas_Object *obj, int x, int y, int w, int h)
 {
-   Eina_Rectangle *rect;
-   Eina_Iterator *it;
-
    API_ENTRY;
 
-   if ((!area) && (!cw->input_area)) return;
-   if (area && cw->input_area && eina_tiler_equal(area, cw->input_area)) return;
-   _e_comp_input_objs_free(cw);
-   evas_object_pass_events_set(cw->obj, 1);
-   if ((!area) || eina_tiler_empty(area))
+   //INF("%d,%d %dx%d", x, y, w, h);
+   if ((cw->input_rect.x == x) && (cw->input_rect.y == y) &&
+       (cw->input_rect.w == w) && (cw->input_rect.h == h)) return;
+   EINA_RECTANGLE_SET(&cw->input_rect, x, y, w, h);
+   if (x || y || (w != cw->ec->client.w) || (h != cw->ec->client.h))
      {
-        E_FREE_FUNC(cw->input_area, eina_tiler_free);
-        E_FREE_FUNC(cw->input_objs, eina_array_free);
-        return;
+        if (!cw->input_obj)
+          {
+             cw->input_obj = evas_object_rectangle_add(e_comp->evas);
+             //e_util_size_debug_set(cw->input_obj, 1);
+             evas_object_name_set(cw->input_obj, "cw->input_obj");
+             evas_object_color_set(cw->input_obj, 0, 0, 0, 0);
+             evas_object_clip_set(cw->input_obj, cw->clip);
+             evas_object_smart_member_add(cw->input_obj, obj);
+          }
+        _e_comp_object_input_rect_update(cw);
+        evas_object_pass_events_set(cw->obj, 1);
+        if (cw->visible) evas_object_show(cw->input_obj);
      }
-   if (cw->input_area)
-     eina_tiler_clear(cw->input_area);
    else
      {
-        cw->input_area = eina_tiler_new(99999, 99999);
-        eina_tiler_tile_size_set(cw->input_area, 1, 1);
+        evas_object_smart_member_del(cw->input_obj);
+        evas_object_freeze_events_set(cw->input_obj, 1);
+        E_FREE_FUNC(cw->input_obj, evas_object_del);
+        evas_object_pass_events_set(cw->obj, 0);
      }
-   if (!cw->input_objs)
-     cw->input_objs = eina_array_new(1);
-   it = eina_tiler_iterator_new(area);
-   EINA_ITERATOR_FOREACH(it, rect)
-     {
-        Evas_Object *o = evas_object_rectangle_add(e_comp->evas);
-        //e_util_size_debug_set(o, 1);
-        evas_object_name_set(o, "cw->input_rect");
-        evas_object_color_set(o, 0, 0, 0, 0);
-        evas_object_clip_set(o, cw->clip);
-        evas_object_smart_member_add(o, obj);
-        evas_object_event_callback_add(o, EVAS_CALLBACK_MOUSE_IN, _e_comp_object_ssd_mouse_in, cw);
-        evas_object_event_callback_add(o, EVAS_CALLBACK_MOUSE_OUT, _e_comp_object_ssd_mouse_out, cw);
-        evas_object_show(o);
-        eina_array_push(cw->input_objs, o);
-        eina_tiler_rect_add(cw->input_area, rect);
-     }
-   eina_iterator_free(it);
-   _e_comp_object_input_rect_update(cw);
 }
 
 E_API Eina_Bool
@@ -3285,16 +3249,12 @@ e_comp_object_coords_inside_input_area(Evas_Object *obj, int x, int y)
 {
    API_ENTRY EINA_FALSE;
 
-   if (cw->input_area)
+   if (cw->input_obj)
      {
-        unsigned int i;
-        Evas_Object *rect;
-        Eina_Array_Iterator it;
-
-        EINA_ARRAY_ITER_NEXT(cw->input_objs, i, rect, it)
-          if (evas_object_pointer_coords_inside_get(rect, x, y)) return EINA_TRUE;
+        int xx, yy, ww, hh;
 
-        return EINA_FALSE;
+        evas_object_geometry_get(cw->input_obj, &xx, &yy, &ww, &hh);
+        return E_INSIDE(x, y, xx, yy, ww, hh);
      }
    if (evas_object_pass_events_get(obj)) return EINA_FALSE;
    return E_INSIDE(x, y, cw->ec->x, cw->ec->y, cw->ec->w, cw->ec->h);
diff --git a/src/bin/e_comp_object.h b/src/bin/e_comp_object.h
index 639d2ad4df..e5bfae2e6a 100644
--- a/src/bin/e_comp_object.h
+++ b/src/bin/e_comp_object.h
@@ -69,7 +69,7 @@ E_API void e_comp_object_signal_emit(Evas_Object *obj, const char *sig, const ch
 E_API void e_comp_object_signal_callback_add(Evas_Object *obj, const char *sig, const char *src, Edje_Signal_Cb cb, const void *data);
 E_API void e_comp_object_signal_callback_del(Evas_Object *obj, const char *sig, const char *src, Edje_Signal_Cb cb);
 E_API void e_comp_object_signal_callback_del_full(Evas_Object *obj, const char *sig, const char *src, Edje_Signal_Cb cb, const void *data);
-E_API void e_comp_object_input_area_set(Evas_Object *obj, const Eina_Tiler *area);
+E_API void e_comp_object_input_area_set(Evas_Object *obj, int x, int y, int w, int h);
 E_API Eina_Bool e_comp_object_coords_inside_input_area(Evas_Object *obj, int x, int y);
 E_API void e_comp_object_damage(Evas_Object *obj, int x, int y, int w, int h);
 E_API Eina_Bool e_comp_object_damage_exists(Evas_Object *obj);
diff --git a/src/bin/e_comp_wl.c b/src/bin/e_comp_wl.c
index f8504d447c..613f4a83dd 100644
--- a/src/bin/e_comp_wl.c
+++ b/src/bin/e_comp_wl.c
@@ -1595,7 +1595,24 @@ _e_comp_wl_surface_state_commit(E_Client *ec, E_Comp_Wl_Surface_State *state)
    /* put state input into surface */
    if (state->input)
      {
-        e_comp_object_input_area_set(ec->frame, state->input);
+        if (!eina_tiler_empty(state->input))
+          {
+             Eina_Rectangle *rect;
+             Eina_Iterator *itr;
+
+             /* This is seriously wrong and results in only the last
+              * rectangle in the region being set, but in the usual
+              * case there's only one rectangle.
+              */
+             itr = eina_tiler_iterator_new(state->input);
+             EINA_ITERATOR_FOREACH(itr, rect)
+               e_comp_object_input_area_set(ec->frame, rect->x, rect->y,
+                                            rect->w, rect->h);
+
+             eina_iterator_free(itr);
+          }
+        else
+          e_comp_object_input_area_set(ec->frame, 1, 1, 0, 0);
 
         eina_tiler_free(state->input);
         state->input = NULL;