From: Shinwoo Kim <cinoo.kim@samsung.com>
Date: Sat, 23 Sep 2017 08:42:55 +0000 (+0900)
Subject: atspi: block event which is not interested in
X-Git-Tag: submit/tizen_3.0/20171019.130042~16
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=952610fbb940c04ef5cfba4fb94052238e4a5ba1;p=platform%2Fupstream%2Felementary.git

atspi: block event which is not interested in

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
---

diff --git a/src/lib/elm_atspi_bridge.c b/src/lib/elm_atspi_bridge.c
index 7c78f7559..5bd49daac 100644
--- a/src/lib/elm_atspi_bridge.c
+++ b/src/lib/elm_atspi_bridge.c
@@ -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
diff --git a/src/lib/elm_interface_atspi_accessible.h b/src/lib/elm_interface_atspi_accessible.h
index e3cf86fcd..bff7a08e8 100644
--- a/src/lib/elm_interface_atspi_accessible.h
+++ b/src/lib/elm_interface_atspi_accessible.h
@@ -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