atspi: block event which is not interested in 08/152008/2
authorShinwoo Kim <cinoo.kim@samsung.com>
Sat, 23 Sep 2017 08:42:55 +0000 (17:42 +0900)
committerGerrit Code Review <gerrit@review.ap-northeast-2.compute.internal>
Wed, 27 Sep 2017 08:55:19 +0000 (08:55 +0000)
The following ATSPI dbus signal is not interested in.
Moreover the following ATSPI dbus signal occurs a lot at init time.
This could cause a performance issue.

 - 'ChildrenChanged' with added child
 - 'ChildrenChanged' with deleted child
 - 'AddAccessible'
 - 'RemoveAccessible'

Change-Id: Iebfb4ef75b5f32bf025e275dd9b548bd3581b2de

src/lib/elm_atspi_bridge.c
src/lib/elm_interface_atspi_accessible.h

index 7c78f7559bd049f92afd62e30b891a4e07654e79..5bd49daaccd41ec57bcb88452f86c0d4e7d7a90c 100644 (file)
@@ -6736,7 +6736,7 @@ static void _embedded_reply_cb(void *data, const Eldbus_Message *msg, Eldbus_Pen
 
    eo_do(proxy, parent = eo_parent_get());
    if (parent)
-     elm_interface_atspi_accessible_children_changed_added_signal_emit(parent, proxy)
+     elm_interface_atspi_accessible_children_changed_added_signal_emit(parent, proxy);
 }
 
 static void
index e3cf86fcda33ce537c54c2a9ec8a4a3544aac458..bff7a08e8c6906984998c70ca7195688f2c9e94e 100644 (file)
@@ -579,20 +579,30 @@ EAPI Elm_Atspi_Relation_Set elm_atspi_relation_set_clone(const Elm_Atspi_Relatio
 /**
  * Emits ATSPI 'ChildrenChanged' dbus signal with added child as argument.
  */
+//TIZEN_ONLY(20170923) Block event which is not interested in
+#define elm_interface_atspi_accessible_children_changed_added_signal_emit(obj, child) /* Empty */
+/*
 #define elm_interface_atspi_accessible_children_changed_added_signal_emit(obj, child) \
    do { \
       Elm_Atspi_Event_Children_Changed_Data atspi_data = { EINA_TRUE, child }; \
    eo_do(ELM_INTERFACE_ATSPI_ACCESSIBLE_MIXIN, elm_interface_atspi_accessible_event_emit(obj, ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_CHILDREN_CHANGED, &atspi_data)); \
    } while(0);
+*/
+//
 
 /**
  * Emits ATSPI 'ChildrenChanged' dbus signal with deleted child as argument.
  */
+//TIZEN_ONLY(20170923) Block event which is not interested in
+#define elm_interface_atspi_accessible_children_changed_del_signal_emit(obj, child) /* Empty */
+/*
 #define elm_interface_atspi_accessible_children_changed_del_signal_emit(obj, child) \
    do { \
       Elm_Atspi_Event_Children_Changed_Data atspi_data = { EINA_FALSE, child }; \
       eo_do(ELM_INTERFACE_ATSPI_ACCESSIBLE_MIXIN, elm_interface_atspi_accessible_event_emit(obj, ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_CHILDREN_CHANGED, &atspi_data)); \
    } while(0);
+*/
+//
 
 /**
  * Emits ATSPI 'ActiveDescendantChanged' dbus signal.
@@ -609,14 +619,24 @@ EAPI Elm_Atspi_Relation_Set elm_atspi_relation_set_clone(const Elm_Atspi_Relatio
 /**
  * Emits ATSPI 'AddAccessible' dbus signal.
  */
+//TIZEN_ONLY(20170923) Block event which is not interested in
+#define elm_interface_atspi_accessible_added(obj) /* Empty */
+/*
 #define elm_interface_atspi_accessible_added(obj) \
    eo_do(ELM_INTERFACE_ATSPI_ACCESSIBLE_MIXIN, elm_interface_atspi_accessible_event_emit(obj, ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_ADDED, NULL));
+*/
+//
 
 /**
  * Emits ATSPI 'RemoveAccessible' dbus signal.
  */
+//TIZEN_ONLY(20170923) Block event which is not interested in
+#define elm_interface_atspi_accessible_removed(obj) /* Empty */
+/*
 #define elm_interface_atspi_accessible_removed(obj) \
    eo_do(ELM_INTERFACE_ATSPI_ACCESSIBLE_MIXIN, elm_interface_atspi_accessible_event_emit(obj, ELM_INTERFACE_ATSPI_ACCESSIBLE_EVENT_REMOVED, NULL));
+*/
+//
 
 #include "elm_interface_atspi_accessible.eo.h"
 #endif