[ATSPI]: Active descendent changed signal not sent issue fix. Due to which

highlight on genlist items was not setting to correct item(highlight jumping) during scroll.

Change-Id: If4e37bafaa50a0a5c61bc67602c87e08574ef902

diff --git a/src/lib/elm_atspi_bridge.c b/src/lib/elm_atspi_bridge.c
index d1b7d054e2dcc356b799093bcac155e6e13aa952..04e08db20593e3702ac4839fe93f2801eaac3f8b 100644 (file)
--- a/src/lib/elm_atspi_bridge.c
+++ b/src/lib/elm_atspi_bridge.c
@@ -4140,7 +4140,9 @@ _active_descendant_changed_signal_send(void *data, Eo *obj, const Eo_Event_Descr
 
    ELM_ATSPI_BRIDGE_DATA_GET_OR_RETURN_VAL(data, pd, EINA_FALSE);
 
-   if (!STATE_TYPE_GET(pd->object_children_broadcast_mask, ATSPI_OBJECT_EVENT_ACTIVE_DESCENDANT_CHANGED))
+   //TIZEN_ONLY(20160914): Issue fix, correct the object pointer for which state was checked.
+   if (!STATE_TYPE_GET(pd->object_broadcast_mask, ATSPI_OBJECT_EVENT_ACTIVE_DESCENDANT_CHANGED))
+   //
      return EINA_FALSE;
 
    eo_do(child, idx = elm_interface_atspi_accessible_index_in_parent_get());

diff --git a/src/lib/elm_gengrid.c b/src/lib/elm_gengrid.c
index d265fef7b1b40f2a9a36a1d1bb49eea5065497fa..5d5ea8708a9cd22684a8838addbea81b86a793e5 100644 (file)
--- a/src/lib/elm_gengrid.c
+++ b/src/lib/elm_gengrid.c
@@ -5765,7 +5765,7 @@ _elm_gengrid_item_elm_interface_atspi_component_highlight_grab(Eo *eo_it, Elm_Ge
       sd->atspi_item_to_highlight = it;//it will be highlighted when realized
 
 ///TIZEN_ONLY(20170717) : expose highlight information on atspi
-   eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_it));
+   elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_it);
 ///
 
    return EINA_TRUE;
@@ -5781,7 +5781,7 @@ _elm_gengrid_item_elm_interface_atspi_component_highlight_clear(Eo *eo_it, Elm_G
 //
    elm_object_accessibility_highlight_set(VIEW(it), EINA_FALSE);
 ///TIZEN_ONLY(20170717) : expose highlight information on atspi
-   eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_it));
+   elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_it);
 ///
 
    return EINA_TRUE;

diff --git a/src/lib/elm_genlist.c b/src/lib/elm_genlist.c
index ebbfd851e83b4aab5278095bef9b2d637601fab5..9633601e70463518fff89253ee7e06e0a2f7b70e 100644 (file)
--- a/src/lib/elm_genlist.c
+++ b/src/lib/elm_genlist.c
@@ -8670,7 +8670,7 @@ _elm_genlist_item_elm_interface_atspi_component_highlight_grab(Eo *eo_it, Elm_Ge
    else
      sd->atspi_item_to_highlight = it;//it will be highlighted when realized
 
-   eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_it));
+   elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_it);
 
    return ret;
 }
@@ -8683,7 +8683,7 @@ _elm_genlist_item_elm_interface_atspi_component_highlight_clear(Eo *eo_it, Elm_G
    if (sd->atspi_item_to_highlight == it)
        sd->atspi_item_to_highlight = NULL;
 
-   eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_it));
+   elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_it);
 
    eo_do_super(eo_it, ELM_GENLIST_ITEM_CLASS, ret = elm_interface_atspi_component_highlight_clear());
    return ret;

diff --git a/src/lib/elm_toolbar.c b/src/lib/elm_toolbar.c
index 95b93db68c325ea2580ef17b1e9623ef316cb09e..1b3488c121245cbf7fa177000d0e4916fe27a13f 100644 (file)
--- a/src/lib/elm_toolbar.c
+++ b/src/lib/elm_toolbar.c
@@ -4148,7 +4148,7 @@ _elm_toolbar_item_elm_interface_atspi_component_highlight_grab(Eo *eo_it, Elm_To
    elm_object_accessibility_highlight_set(VIEW(it), EINA_TRUE);
 
 ///TIZEN_ONLY(20170717) : expose highlight information on atspi
-   eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_it));
+   elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_it);
 ///
    return EINA_TRUE;
 }
@@ -4158,7 +4158,7 @@ _elm_toolbar_item_elm_interface_atspi_component_highlight_clear(Eo *eo_it, Elm_T
 {
    elm_object_accessibility_highlight_set(VIEW(it), EINA_FALSE);
 ///TIZEN_ONLY(20170717) : expose highlight information on atspi
-   eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_it));
+   elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_it);
 ///
    return EINA_TRUE;
 }

diff --git a/src/mobile_lib/elm_genlist.c b/src/mobile_lib/elm_genlist.c
index 0ec177afe59e35b5b97ef2d18d61416b7d841db2..a9122c5d2bd5923614ee1e7a2e1c39a1dc931018 100644 (file)
--- a/src/mobile_lib/elm_genlist.c
+++ b/src/mobile_lib/elm_genlist.c
@@ -2575,7 +2575,7 @@ static void _item_focused(Elm_Gen_Item *it, Elm_Genlist_Item_Scrollto_Type type)
    sd->focused_item = it;
    evas_object_smart_callback_call(WIDGET(it), SIG_ITEM_FOCUSED, EO_OBJ(it));
    if (_elm_config->atspi_mode)
-     eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, EO_OBJ(it)));
+     elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), EO_OBJ(it));
 }
 
 #ifndef ELM_FEATURE_WEARABLE
@@ -2667,7 +2667,7 @@ _item_highlight(Elm_Gen_Item *it)
 //****************************
    evas_object_smart_callback_call(WIDGET(it), SIG_HIGHLIGHTED, EO_OBJ(it));
    if (_elm_config->atspi_mode)
-     eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, EO_OBJ(it)));
+     elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), EO_OBJ(it));
 }
 
 static void
@@ -6665,7 +6665,7 @@ _elm_genlist_item_expanded_set(Eo *eo_item EINA_UNUSED, Elm_Gen_Item *it, Eina_B
           edje_object_signal_emit(VIEW(it), SIGNAL_EXPANDED, "elm");
         evas_object_smart_callback_call(WIDGET(it), SIG_EXPANDED, EO_OBJ(it));
         if (_elm_config->atspi_mode)
-          eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_item));
+          elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_item);
      }
    else
      {
@@ -6673,7 +6673,7 @@ _elm_genlist_item_expanded_set(Eo *eo_item EINA_UNUSED, Elm_Gen_Item *it, Eina_B
           edje_object_signal_emit(VIEW(it), SIGNAL_CONTRACTED, "elm");
         evas_object_smart_callback_call(WIDGET(it), SIG_CONTRACTED, EO_OBJ(it));
         if (_elm_config->atspi_mode)
-          eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_item));
+          elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_item);
      }
 }
 
@@ -8060,7 +8060,7 @@ _elm_genlist_item_elm_interface_atspi_component_highlight_grab(Eo *eo_it, Elm_Ge
        sd->atspi_item_to_highlight = it;//it will be highlighted when realized
 
 ///TIZEN_ONLY(20170717) : expose highlight information on atspi
-   eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_it));
+   elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_it);
 ///
 
    return EINA_TRUE;
@@ -8074,7 +8074,7 @@ _elm_genlist_item_elm_interface_atspi_component_highlight_clear(Eo *eo_it, Elm_G
        sd->atspi_item_to_highlight = NULL;
    elm_object_accessibility_highlight_set(VIEW(it), EINA_FALSE);
 ///TIZEN_ONLY(20170717) : expose highlight information on atspi
-   eo_do(WIDGET(it), eo_event_callback_call(ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ACTIVE_DESCENDANT_CHANGED, eo_it));
+   elm_interface_atspi_accessible_active_descendant_changed_signal_emit(WIDGET(it), eo_it);
 ///
 
    return EINA_TRUE;