From: Shinwoo Kim <cinoo.kim@samsung.com>
Date: Wed, 20 Dec 2017 07:58:38 +0000 (+0530)
Subject: elm: access: emit signal after atspi bridge is connected
X-Git-Tag: submit/sandbox/upgrade/efl120/20180319.053334~655
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=31e426a9f78820e487055fe1e973fe3fab497ba1;p=platform%2Fupstream%2Fefl.git

elm: access: emit signal after atspi bridge is connected

This patch set emitting popup SHOWING signal, and window ACTIVE signal after
the atspi bridge is connectted.

The SHOWING signal of popup does not work properly, if the popup shows before
the atspi bridge is not connected. In this case, the screen-reader cannot read
popup information. So the SHOWING signal should be emitted after the connection
is established.

The ACTIVE signal does not work properly in the following case.
There are two windows show in one process at the same time. In this case,
the ACTIVE signal was not emitted because window_activated_broadcast_needed is
set to EINA_FALSE. This would be a bug.

orig: fe8b20fac57caa39c0f801d3819e688f391c1831

Change-Id: I7e88b20c1d31ecc41410d8d81159654ce637e133
Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
---

diff --git a/src/lib/elementary/efl_ui_win.c b/src/lib/elementary/efl_ui_win.c
index b72ea47..35fe4cb 100644
--- a/src/lib/elementary/efl_ui_win.c
+++ b/src/lib/elementary/efl_ui_win.c
@@ -4180,11 +4180,10 @@ _elm_win_atspi(Eina_Bool is_atspi)
 {
    const Eina_List *l;
    Evas_Object *obj;
+   Efl_Access_State_Set ss;
 
    EINA_LIST_FOREACH(_elm_win_list, l, obj)
      {
-        elm_widget_atspi(obj, is_atspi);
-
         if (!is_atspi)
           {
              _access_socket_proxy_unref(obj);
@@ -4192,7 +4191,14 @@ _elm_win_atspi(Eina_Bool is_atspi)
         else
           {
              _access_socket_proxy_listen(obj);
+             ss = efl_access_state_set_get(obj);
+             if (STATE_TYPE_GET(ss, EFL_ACCESS_STATE_ACTIVE))
+               {
+                  efl_access_window_activated_signal_emit(obj);
+                  efl_access_state_changed_signal_emit(obj, EFL_ACCESS_STATE_ACTIVE, EINA_TRUE);
+               }
           }
+        elm_widget_atspi(obj, is_atspi);
      }
 }
 //
diff --git a/src/lib/elementary/elm_popup.eo b/src/lib/elementary/elm_popup.eo
index 826b20e..207306e 100644
--- a/src/lib/elementary/elm_popup.eo
+++ b/src/lib/elementary/elm_popup.eo
@@ -181,6 +181,7 @@ class Elm.Popup (Efl.Ui.Layout, Efl.Ui.Focus.Layer, Efl.Access.Widget.Action)
       //register/unregister access objects accordingly.
       // TIZEN_ONLY(20170516): connect to at-spi dbus based on org.a11y.Status.IsEnabled property
       Elm.Widget.screen_reader;
+      Elm.Widget.atspi;
       //
       //
       Efl.Layout.Signal.signal_emit;