2009-04-23 Mark Doffman <mark.doffman@codethink.co.uk>

        Fix to update the cache on state-change events.

diff --git a/atk-adaptor/accessible-register.c b/atk-adaptor/accessible-register.c
index d143aa7..c6c7328 100644 (file)
--- a/atk-adaptor/accessible-register.c
+++ b/atk-adaptor/accessible-register.c
@@ -385,6 +385,12 @@ atk_dbus_desktop_object_path ()
 
 /*---------------------------------------------------------------------------*/
 
 
 /*---------------------------------------------------------------------------*/
 

+typedef gboolean (*TreeUpdateAction) (GSignalInvocationHint *signal_hint,
+                                      guint                  n_param_values,
+                                      const GValue          *param_values,
+                                      gpointer               data,
+                                      AtkObject             *accessible);
+

 /*
  * Events are not evaluated for non-registered accessibles.
  *
 /*
  * Events are not evaluated for non-registered accessibles.
  *


@@ -394,22 +400,22 @@ atk_dbus_desktop_object_path ()
  * When a parent is changed the subtree is de-registered
  * if the parent is not attached to the root accessible.
  */
  * When a parent is changed the subtree is de-registered
  * if the parent is not attached to the root accessible.
  */

+/* TODO Turn this function into a macro? */

 static gboolean
 static gboolean

-tree_update_listener (GSignalInvocationHint *signal_hint,
-                      guint                  n_param_values,
-                      const GValue          *param_values,
-                      gpointer               data)
+tree_update_wrapper (GSignalInvocationHint *signal_hint,
+                     guint                  n_param_values,
+                     const GValue          *param_values,
+                     gpointer               data,
+                     TreeUpdateAction       action)

 {
   AtkObject *accessible;
 {
   AtkObject *accessible;

-  AtkPropertyValues *values;
-  const gchar *pname = NULL;

 
   g_static_rec_mutex_lock (&registration_mutex);
 
   /* Ensure that only registered accessibles
    * have their signals processed.
    */
 
   g_static_rec_mutex_lock (&registration_mutex);
 
   /* Ensure that only registered accessibles
    * have their signals processed.
    */

-  accessible = g_value_get_object (&param_values[0]);
+  accessible = ATK_OBJECT(g_value_get_object (&param_values[0]));

   g_return_val_if_fail (ATK_IS_OBJECT (accessible), TRUE);
 
   if (object_to_ref (accessible))
   g_return_val_if_fail (ATK_IS_OBJECT (accessible), TRUE);
 
   if (object_to_ref (accessible))


@@ -420,7 +426,44 @@ tree_update_listener (GSignalInvocationHint *signal_hint,
 
       g_debug ("AT-SPI: Tree update listener");
 #endif
 
       g_debug ("AT-SPI: Tree update listener");
 #endif

+      action (signal_hint, n_param_values, param_values, data, accessible);
+
+      recursion_check_unset ();
+    }
+
+  g_static_rec_mutex_unlock (&registration_mutex);
+
+  return TRUE;
+}
+
+static gboolean
+tree_update_state_action (GSignalInvocationHint *signal_hint,
+                          guint                  n_param_values,
+                          const GValue          *param_values,
+                          gpointer               data,
+                          AtkObject             *accessible)
+{
+      update_accessible (accessible);
+}
+
+static gboolean
+tree_update_state_listener (GSignalInvocationHint *signal_hint,
+                            guint                  n_param_values,
+                            const GValue          *param_values,
+                            gpointer               data)
+{
+      tree_update_wrapper (signal_hint, n_param_values, param_values, data, tree_update_state_action);
+}

 
 

+static gboolean
+tree_update_property_action (GSignalInvocationHint *signal_hint,
+                             guint                  n_param_values,
+                             const GValue          *param_values,
+                             gpointer               data,
+                             AtkObject             *accessible)
+{
+      AtkPropertyValues *values;
+      const gchar *pname = NULL;

 
       values = (AtkPropertyValues*) g_value_get_pointer (&param_values[1]);
       pname = values[0].property_name;
 
       values = (AtkPropertyValues*) g_value_get_pointer (&param_values[1]);
       pname = values[0].property_name;


@@ -431,47 +474,26 @@ tree_update_listener (GSignalInvocationHint *signal_hint,
           update_accessible (accessible);
         }
       /* Parent value us updated by child-add signal of parent object */
           update_accessible (accessible);
         }
       /* Parent value us updated by child-add signal of parent object */

-
-      recursion_check_unset ();
-    }
-
-  g_static_rec_mutex_unlock (&registration_mutex);
-
-  return TRUE;

 }
 
 }
 

-/*
- * Events are not evaluated for non registered accessibles.
- *
- * When the children of a registered accessible are changed
- * the subtree, rooted at the child is registered.
- */

 static gboolean
 static gboolean

-tree_update_children_listener (GSignalInvocationHint *signal_hint,
+tree_update_property_listener (GSignalInvocationHint *signal_hint,

                                guint                  n_param_values,
                                const GValue          *param_values,
                                gpointer               data)
 {
                                guint                  n_param_values,
                                const GValue          *param_values,
                                gpointer               data)
 {

-  AtkObject *accessible;
-  const gchar *detail = NULL;
-  AtkObject *child;
-
-  g_static_rec_mutex_lock (&registration_mutex);
-
-  /* Ensure that only registered accessibles
-   * have their signals processed.
-   */
-  accessible = g_value_get_object (&param_values[0]);
-  g_return_val_if_fail (ATK_IS_OBJECT (accessible), TRUE);
-
-  if (object_to_ref (accessible))
-    {
-#ifdef SPI_ATK_DEBUG
-      if (recursion_check_and_set ())
-          g_warning ("AT-SPI: Recursive use of registration module");
+      tree_update_wrapper (signal_hint, n_param_values, param_values, data, tree_update_property_action);
+}

 
 

-      g_debug ("AT-SPI: Tree update children listener");
-#endif
+static gboolean
+tree_update_children_action (GSignalInvocationHint *signal_hint,
+                             guint                  n_param_values,
+                             const GValue          *param_values,
+                             gpointer               data,
+                             AtkObject             *accessible)
+{
+      const gchar *detail = NULL;
+      AtkObject *child;

 
       if (signal_hint->detail)
           detail = g_quark_to_string (signal_hint->detail);
 
       if (signal_hint->detail)
           detail = g_quark_to_string (signal_hint->detail);


@@ -492,15 +514,19 @@ tree_update_children_listener (GSignalInvocationHint *signal_hint,
           register_subtree (child);
           update_accessible (accessible);
         }
           register_subtree (child);
           update_accessible (accessible);
         }

+}

 
 

-      recursion_check_unset ();
-    }
-
-  g_static_rec_mutex_unlock (&registration_mutex);
-
-  return TRUE;
+static gboolean
+tree_update_children_listener (GSignalInvocationHint *signal_hint,
+                               guint                  n_param_values,
+                               const GValue          *param_values,
+                               gpointer               data)
+{
+      tree_update_wrapper (signal_hint, n_param_values, param_values, data, tree_update_children_action);

 }
 
 }
 

+/*---------------------------------------------------------------------------*/
+

 static void
 spi_atk_register_toplevel_added (AtkObject *accessible,
                                  guint     index,
 static void
 spi_atk_register_toplevel_added (AtkObject *accessible,
                                  guint     index,


@@ -579,8 +605,9 @@ atk_dbus_initialize (AtkObject *root)
 
   register_subtree (root);
 
 
   register_subtree (root);
 

-  atk_add_global_event_listener (tree_update_listener, "Gtk:AtkObject:property-change");
+  atk_add_global_event_listener (tree_update_property_listener, "Gtk:AtkObject:property-change");

   atk_add_global_event_listener (tree_update_children_listener, "Gtk:AtkObject:children-changed");
   atk_add_global_event_listener (tree_update_children_listener, "Gtk:AtkObject:children-changed");

+  atk_add_global_event_listener (tree_update_state_listener, "Gtk:AtkObject:state-change");

 
   g_signal_connect (root,
                     "children-changed::add",
 
   g_signal_connect (root,
                     "children-changed::add",

diff --git a/atk-adaptor/bridge.c b/atk-adaptor/bridge.c
index b99f982..eb2fa75 100644 (file)
--- a/atk-adaptor/bridge.c
+++ b/atk-adaptor/bridge.c
@@ -311,12 +311,12 @@ adaptor_init (gint *argc, gchar **argv[])
   spi_initialize_text (accpath);
   spi_initialize_value (accpath);
 
   spi_initialize_text (accpath);
   spi_initialize_value (accpath);
 

-  /* Register methods to send D-Bus signals on certain ATK events */
-  spi_atk_register_event_listeners ();
-

   /* Initialize the AtkObject registration */
   atk_dbus_initialize (atk_adaptor_app_data->root);
 
   /* Initialize the AtkObject registration */
   atk_dbus_initialize (atk_adaptor_app_data->root);
 

+  /* Register methods to send D-Bus signals on certain ATK events */
+  spi_atk_register_event_listeners ();
+

   /* Register this app by sending a signal out to AT-SPI registry daemon */
   register_application (atk_adaptor_app_data);
 
   /* Register this app by sending a signal out to AT-SPI registry daemon */
   register_application (atk_adaptor_app_data);
 

diff --git a/atk-adaptor/event.c b/atk-adaptor/event.c
index 3eababe..a201b37 100644 (file)
--- a/atk-adaptor/event.c
+++ b/atk-adaptor/event.c
@@ -701,7 +701,9 @@ spi_atk_register_event_listeners (void)
   add_signal_listener (document_event_listener,               "Gtk:AtkDocument:load-complete");
   add_signal_listener (document_event_listener,               "Gtk:AtkDocument:reload");
   add_signal_listener (document_event_listener,               "Gtk:AtkDocument:load-stopped");
   add_signal_listener (document_event_listener,               "Gtk:AtkDocument:load-complete");
   add_signal_listener (document_event_listener,               "Gtk:AtkDocument:reload");
   add_signal_listener (document_event_listener,               "Gtk:AtkDocument:load-stopped");

+  /* TODO Fake this event on the client side */

   add_signal_listener (state_event_listener,                  "Gtk:AtkObject:state-change");
   add_signal_listener (state_event_listener,                  "Gtk:AtkObject:state-change");

+  /* TODO */

   add_signal_listener (active_descendant_event_listener,      "Gtk:AtkObject:active-descendant-changed");
   add_signal_listener (bounds_event_listener,                 "Gtk:AtkComponent:bounds-changed");
   add_signal_listener (text_selection_changed_event_listener, "Gtk:AtkText:text-selection-changed");
   add_signal_listener (active_descendant_event_listener,      "Gtk:AtkObject:active-descendant-changed");
   add_signal_listener (bounds_event_listener,                 "Gtk:AtkComponent:bounds-changed");
   add_signal_listener (text_selection_changed_event_listener, "Gtk:AtkText:text-selection-changed");