Changes to enable new introspection format.
[platform/core/uifw/at-spi2-atk.git] / atk-bridge / bridge.c
index 86bbaa5..ec6ddd0 100644 (file)
@@ -2,8 +2,8 @@
  * AT-SPI - Assistive Technology Service Provider Interface
  * (Gnome Accessibility Project; http://developer.gnome.org/projects/gap)
  *
- * Copyright 2001, 2002 Sun Microsystems Inc.,
- * Copyright 2001, 2002 Ximian, Inc.
+ * Copyright 2001, 2002, 2003 Sun Microsystems Inc.,
+ * Copyright 2001, 2002, 2003 Ximian, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * Boston, MA 02111-1307, USA.
  */
 
+#include "config.h"
+#include "dbus/dbus-glib.h"
+
+#include <X11/Xlib.h>
+#include <X11/Xatom.h>
+#include <string.h>
 #include <stdio.h>
+#include <unistd.h>
 #include <stdlib.h>
 #include <stdarg.h>
-#include <libbonobo.h>
-#include <orbit/orbit.h>
 #include <atk/atk.h>
 #include <atk/atkobject.h>
 #include <atk/atknoopobject.h>
-#include <libspi/Accessibility.h>
 #include "accessible.h"
-#include "application.h"
-
-#include <bonobo-activation/bonobo-activation-register.h>
 
 #undef SPI_BRIDGE_DEBUG
 
 #define DBG(a,b) if(_dbg>=(a))b
 
-static int _dbg = 0;
-static char *spi_nil_string = "";
+#define bridge_threads_leave() \
+  if (!during_init_shutdown && !g_main_context_is_owner (NULL)) atk_misc_threads_leave(misc);
+#define bridge_threads_enter() \
+  if (!during_init_shutdown && !g_main_context_is_owner (NULL)) atk_misc_threads_enter(misc);
 
-static CORBA_Environment ev;
-static Accessibility_Registry registry = NULL;
-static SpiApplication *this_app = NULL;
+typedef struct _SpiAppData SpiAppData;
+struct _SpiAppData
+{
+  AtkObject *root;
+  DRouteData droute;
+};
+
+int _dbg = 0;
+static const char *registry = NULL;
+static char *device_event_controller = NULL;
+static SpiAppData *this_app = NULL;
 static gboolean registry_died = FALSE;
 static gboolean atk_listeners_registered = FALSE;
-static guint toplevel_handler;
-
-/* NOT YET USED
-   static GQuark atk_quark_property_changed_name;
-   static GQuark atk_quark_property_changed_description;
-   static GQuark atk_quark_property_changed_parent;
-   static GQuark atk_quark_property_changed_role;
-   static GQuark atk_quark_property_changed_table_caption;
-   static GQuark atk_quark_property_changed_table_column_description;
-   static GQuark atk_quark_property_changed_table_row_description;
-   static guint atk_signal_property_changed;
-*/
+static gint toplevels = 0;
+static gboolean exiting = FALSE;
+static AtkMisc *misc = NULL;
+static gboolean during_init_shutdown = TRUE;
 
 static guint atk_signal_text_changed;
-static guint atk_signal_child_changed;
+static guint atk_signal_children_changed;
+static guint atk_signal_active_descendant_changed;
+static guint atk_signal_text_selection_changed;
 
 /* NOT YET USED
-   static guint atk_signal_text_selection_changed;
-   static guint atk_signal_active_descendant_changed;
    static guint atk_signal_row_reordered;
    static guint atk_signal_row_inserted;
    static guint atk_signal_row_deleted;
@@ -74,26 +77,39 @@ static guint atk_signal_child_changed;
    static guint atk_signal_column_deleted;
 */
 
-static Accessibility_Registry spi_atk_bridge_get_registry (void);
-static void     spi_atk_bridge_do_registration         (void);
+static guint atk_signal_link_selected;
+static guint atk_signal_bounds_changed;
+
+static const char *spi_atk_bridge_get_registry (void);
+static gboolean spi_atk_bridge_do_registration         (void);
 static void     spi_atk_bridge_toplevel_added          (AtkObject             *object,
                                                         guint                 index,
                                                         AtkObject             *child);
+static void     spi_atk_bridge_toplevel_removed        (AtkObject             *object,
+                                                        guint                 index,
+                                                        AtkObject             *child);
 
 static void     spi_atk_bridge_exit_func               (void);
 static void     spi_atk_register_event_listeners       (void);
 static void     spi_atk_bridge_focus_tracker           (AtkObject             *object);
-static void     spi_atk_bridge_register_application    (Accessibility_Registry registry);
+static gchar   *spi_atk_bridge_get_registry_ior        (void);
+static void     spi_atk_bridge_register_application    (const char *registry);
 static gboolean spi_atk_bridge_property_event_listener (GSignalInvocationHint *signal_hint,
                                                        guint                  n_param_values,
                                                        const GValue          *param_values,
                                                        gpointer               data);
+
 static gboolean
 spi_atk_bridge_window_event_listener (GSignalInvocationHint *signal_hint,
                                guint n_param_values,
                                const GValue *param_values,
                                gpointer data);
 static gboolean
+spi_atk_bridge_document_event_listener (GSignalInvocationHint *signal_hint,
+                                guint n_param_values,
+                                const GValue *param_values,
+                                gpointer data);
+static gboolean
 spi_atk_bridge_state_event_listener (GSignalInvocationHint *signal_hint,
                                     guint n_param_values,
                                     const GValue *param_values,
@@ -104,12 +120,16 @@ static gboolean spi_atk_bridge_signal_listener         (GSignalInvocationHint *s
                                                        gpointer               data);
 static gint     spi_atk_bridge_key_listener            (AtkKeyEventStruct     *event,
                                                        gpointer               data);
+static void     spi_atk_tidy_windows                   (void);
+static void     deregister_application                 ();
+static void reinit_register_vars (void);
 
 /* For automatic libgnome init */
 extern void gnome_accessibility_module_init     (void);
 extern void gnome_accessibility_module_shutdown (void);
 
 static int     atk_bridge_initialized = FALSE;
+static pid_t   atk_bridge_pid = 0;
 static guint   atk_bridge_focus_tracker_id = 0;
 static guint   atk_bridge_key_event_listener_id = 0;
 static GArray *listener_ids = NULL;
@@ -124,73 +144,160 @@ extern void gnome_accessibility_module_shutdown (void);
 static void
 spi_atk_bridge_init_event_type_consts ()
 {
-  atk_signal_child_changed = g_signal_lookup ("child_changed", 
+  static gboolean done = FALSE;
+
+  if (done)
+    return;
+
+  atk_signal_children_changed = g_signal_lookup ("children_changed", 
                                              ATK_TYPE_OBJECT);
   atk_signal_text_changed = g_signal_lookup ("text_changed", 
                                             ATK_TYPE_TEXT);
+  atk_signal_bounds_changed = g_signal_lookup ("bounds_changed", 
+                                             ATK_TYPE_COMPONENT);
+  atk_signal_active_descendant_changed = 
+         g_signal_lookup ("active_descendant_changed", 
+                         ATK_TYPE_OBJECT); 
+  atk_signal_link_selected = g_signal_lookup ("link_selected", 
+                                             ATK_TYPE_HYPERTEXT);
+  atk_signal_text_selection_changed = g_signal_lookup ("text_selection_changed", 
+                                             ATK_TYPE_TEXT);
+  done = TRUE;
+}
+
+static gboolean
+post_init (void)
+{
+  during_init_shutdown = FALSE;
+  return FALSE;
+}
+
+static DBusObjectPathVTable droute_vtable =
+{
+  NULL,
+  &droute_message,
+  NULL, NULL, NULL, NULL
+};
+
+static SpiAppData *
+spi_app_init (AtkObject *root)
+{
+  DBusError error;
+  dbus_error_init(&error);
+  SpiAppData *ad = (SpiAppData *)calloc(sizeof(SpiAppData), 1);
+  if (!ad) return NULL;
+  ad->root = root;
+  ad->droute.bus = dbus_bus_get(DBUS_BUS_SESSION, &error);
+  if (!ad->droute.bus)
+  {
+    g_warning("Couldn't connect to dbus: %s\n", error.message);
+    free(ad);
+    return NULL;
+  }
+  //dbus_connection_set_exit_on_disconnect(ad->droute.bus, FALSE);
+  //dbus_bus_register(ad->droute.bus, &error);
+  spi_dbus_initialize (&ad->droute);
+  /* Below line for testing -- it should be removed once at-spi-registryd is working */
+  if (dbus_bus_request_name(ad->droute.bus, "test.atspi.tree", 0, &error)) printf("Got test name.\n");
+  spi_register_tree_object(ad->droute.bus, "/org/freedesktop/atspi/tree");
+  if (!dbus_connection_try_register_fallback (ad->droute.bus, "/org/freedesktop/atspi/accessible/", &droute_vtable, &ad->droute, &error))
+  {
+    g_warning("Couldn't register droute.\n");
+  }
+  dbus_connection_setup_with_g_main(ad->droute.bus, g_main_context_default());
+  return ad;
 }
 
 static int
 atk_bridge_init (gint *argc, gchar **argv[])
 {
   const char *debug_env_string = g_getenv ("AT_SPI_DEBUG");
+  gchar *fname;
+  gboolean success = FALSE;
 
   if (atk_bridge_initialized)
     {
       return 0;
     }
   atk_bridge_initialized = TRUE;
+  atk_bridge_pid = getpid ();
 
-  if (debug_env_string)
-    _dbg = (int) g_ascii_strtod (debug_env_string, NULL);
+  misc = atk_misc_get_instance();
 
-  if (!bonobo_init (argc, argv ? *argv : NULL))
+  if (g_getenv ("ATK_BRIDGE_REDIRECT_LOG"))
     {
-      g_error ("Could not initialize Bonobo");
+      fname = g_strconcat ("/tmp/", g_get_prgname (), ".at-spi-log", NULL);
+      /* make sure we're not being redirected - security issue */
+      if (!g_file_test (fname, G_FILE_TEST_IS_SYMLINK))
+         freopen (fname, "w", stderr);
+      g_free (fname);
     }
 
+  if (debug_env_string) 
+      _dbg = (int) g_ascii_strtod (debug_env_string, NULL);
+
+  /* Connect to dbus */
+  this_app = spi_app_init (atk_get_root ());
+
   /*
    * We only want to enable the bridge for top level
    * applications, we detect bonobo components by seeing
    * if they were activated with the intention of extracting
    * an impl. by IID - very solid.
    */
+#ifdef WITH_BONOBO
+  // TODO: Figure out if this is still needed
   if (bonobo_activation_iid_get ())
+#else
+  if (0)
+#endif
     {
       DBG (1, g_message ("Found Bonobo component\n"));
-      toplevel_handler = g_signal_connect (atk_get_root (), 
-                                           "children-changed::add",
-                                           (GCallback) spi_atk_bridge_toplevel_added, 
-                                           NULL);
+      g_signal_connect (atk_get_root (), 
+                        "children-changed::add",
+                        (GCallback) spi_atk_bridge_toplevel_added, 
+                        NULL);
+      g_signal_connect (atk_get_root (), 
+                        "children-changed::remove",
+                        (GCallback) spi_atk_bridge_toplevel_removed, 
+                        NULL);
+      /* in this case we redefine 'success' to mean 'registry is present' */
+      success = (spi_atk_bridge_get_registry () != NULL);
     }
   else
     {
-      spi_atk_bridge_do_registration ();
+      success = spi_atk_bridge_do_registration ();
+    }
+  /*
+   * we must emit events even if we are not registered as a
+   * full-fledged app; See bugzilla #400709.
+   */
+  if (success) 
+    {
+      spi_atk_register_event_listeners ();
+      spi_atk_bridge_init_event_type_consts ();
     }
-  spi_atk_bridge_init_event_type_consts ();
+  else
+    {
+      atk_bridge_initialized = FALSE;
+    }
+  g_idle_add (post_init, NULL);
 
   return 0;
 }
 
-
-static void
+static gboolean
 spi_atk_bridge_do_registration (void)
 {
-  CORBA_Environment ev;
-
-  CORBA_exception_init(&ev);
-
-  if (spi_atk_bridge_get_registry () == CORBA_OBJECT_NIL)
+  if (spi_atk_bridge_get_registry () == NULL)
     {
-      g_error ("Could not locate registry");
+      g_warning ("Could not locate registry");
+      return FALSE;
     }
 
-  bonobo_activate ();
-
   /* Create the accessible application server object */
-
-  this_app = spi_application_new (atk_get_root ());
+  if (this_app == NULL)
+    this_app = spi_app_init (atk_get_root ());
 
   DBG (1, g_message ("About to register application\n"));
 
@@ -199,7 +306,7 @@ spi_atk_bridge_do_registration (void)
   g_atexit (spi_atk_bridge_exit_func);
 
   DBG (1, g_message ("Application registered & listening\n"));
-
+  return TRUE;
 }
 
 static void
@@ -207,45 +314,116 @@ spi_atk_bridge_toplevel_added (AtkObject *object,
                                guint     index,
                                AtkObject *child)
 {
-  g_signal_handler_disconnect (object, toplevel_handler);
-  spi_atk_bridge_do_registration ();
+  if (toplevels == 0)
+    {
+      spi_atk_bridge_do_registration ();
+    }
+  toplevels++;
 }
 
 static void
-spi_atk_bridge_register_application (Accessibility_Registry registry)
+spi_atk_bridge_toplevel_removed (AtkObject *object,
+                                 guint     index,
+                                 AtkObject *child)
 {
-  Accessibility_Registry_registerApplication (spi_atk_bridge_get_registry (),
-                                              BONOBO_OBJREF (this_app),
-                                              &ev);
-  spi_atk_register_event_listeners ();
+  toplevels--;
+  if (toplevels == 0)
+    {
+      deregister_application (this_app);
+      reinit_register_vars ();
+    }
+  if (toplevels < 0)
+    {
+      g_warning ("More toplevels removed than added\n");
+      toplevels = 0;
+    }
 }
 
-static Accessibility_Registry
-spi_atk_bridge_get_registry ()
+static void
+spi_atk_bridge_register_application (const char *registry)
 {
-  CORBA_Environment ev;
+  bridge_threads_leave ();
+  // TODO: register
+  bridge_threads_enter ();
+}
 
-  if (registry_died || (registry == NULL)) {
-         CORBA_exception_init (&ev);
-         if (registry_died) 
-           DBG (1, g_warning ("registry died! restarting..."));
-         registry = bonobo_activation_activate_from_id (
-                 "OAFIID:Accessibility_Registry:1.0", 0, NULL, &ev);
-         
-         if (ev._major != CORBA_NO_EXCEPTION)
+/* 
+ * Returns a 'canonicalized' value for DISPLAY,
+ * with the screen number stripped off if present.
+ */
+static const gchar*
+spi_display_name (void)
+{
+    static const char *canonical_display_name = NULL;
+    if (!canonical_display_name)
+      {
+        const gchar *display_env = g_getenv ("AT_SPI_DISPLAY");
+       if (!display_env)
          {
-                 g_error ("Accessibility app error: exception during "
-                          "registry activation from id: %s\n",
-                          CORBA_exception_id (&ev));
-                 CORBA_exception_free (&ev);
+           display_env = g_getenv ("DISPLAY");
+           if (!display_env || !display_env[0]) 
+               canonical_display_name = ":0";
+           else
+             {
+               gchar *display_p, *screen_p;
+               canonical_display_name = g_strdup (display_env);
+               display_p = strrchr (canonical_display_name, ':');
+               screen_p = strrchr (canonical_display_name, '.');
+               if (screen_p && display_p && (screen_p > display_p))
+                 {
+                   *screen_p = '\0';
+                 }
+             }
          }
-         
-         if (registry_died && registry) {
-                 registry_died = FALSE;
-                 spi_atk_bridge_register_application (registry);
+       else
+         {
+           canonical_display_name = display_env;
          }
-  }
-  return registry;
+      }
+    return canonical_display_name;
+}
+
+static     Display *bridge_display = NULL;
+
+static gchar *
+spi_atk_bridge_get_registry_ior (void)
+{
+     
+     Atom AT_SPI_IOR;
+     Atom actual_type;
+     int actual_format;
+     unsigned char *data = NULL;  
+     unsigned long nitems;
+     unsigned long leftover;
+     if (!bridge_display) 
+       bridge_display = XOpenDisplay (spi_display_name ());
+
+     AT_SPI_IOR = XInternAtom (bridge_display, "AT_SPI_IOR", False); 
+     XGetWindowProperty(bridge_display, 
+                       XDefaultRootWindow (bridge_display),
+                       AT_SPI_IOR, 0L, 
+                       (long)BUFSIZ, False, 
+                       (Atom) 31, &actual_type, &actual_format,
+                       &nitems, &leftover, &data);
+     if (data == NULL)
+         g_warning (_("AT_SPI_REGISTRY was not started at session startup."));
+     
+     return (gchar *) data;
+     
+}
+
+
+static const char *
+spi_atk_bridge_get_registry (void)
+{
+  // TODO: check for registry dying, as the old code attempted to do
+  return "org.freedesktop.atspi.registry";
+}
+
+static const char *
+spi_atk_bridget_get_dec (void)
+{
+  return "/dec";
 }
 
 int
@@ -277,7 +455,12 @@ spi_atk_register_event_listeners (void)
   AtkObject *bo = atk_no_op_object_new (ao);
 
 
-  if (atk_listeners_registered) return;
+  if (atk_listeners_registered) 
+    {
+      g_object_unref (G_OBJECT (bo));
+      g_object_unref (ao);
+      return;
+    }
 
   atk_listeners_registered = TRUE;
 
@@ -311,12 +494,23 @@ spi_atk_register_event_listeners (void)
   id = atk_add_global_event_listener (spi_atk_bridge_window_event_listener,
                                      "window:deactivate");
   g_array_append_val (listener_ids, id);
+  id = atk_add_global_event_listener (spi_atk_bridge_document_event_listener,
+                                     "Gtk:AtkDocument:load-complete");
+  g_array_append_val (listener_ids, id);
+  id = atk_add_global_event_listener (spi_atk_bridge_document_event_listener,
+                                     "Gtk:AtkDocument:reload");
+  g_array_append_val (listener_ids, id);
+  id = atk_add_global_event_listener (spi_atk_bridge_document_event_listener,
+                                     "Gtk:AtkDocument:load-stopped");
+  g_array_append_val (listener_ids, id);
   id = atk_add_global_event_listener (spi_atk_bridge_state_event_listener,
                                      "Gtk:AtkObject:state-change");
   g_array_append_val (listener_ids, id);
 
   add_signal_listener ("Gtk:AtkObject:children-changed");
   add_signal_listener ("Gtk:AtkObject:visible-data-changed");
+  add_signal_listener ("Gtk:AtkObject:active-descendant-changed");
+  add_signal_listener ("Gtk:AtkComponent:bounds-changed");
   add_signal_listener ("Gtk:AtkSelection:selection-changed");
   add_signal_listener ("Gtk:AtkText:text-selection-changed");
   add_signal_listener ("Gtk:AtkText:text-changed");
@@ -328,6 +522,7 @@ spi_atk_register_event_listeners (void)
   add_signal_listener ("Gtk:AtkTable:column-reordered");
   add_signal_listener ("Gtk:AtkTable:column-deleted");
   add_signal_listener ("Gtk:AtkTable:model-changed");
+  add_signal_listener ("Gtk:AtkHypertext:link-selected");
 /*
  * May add the following listeners to implement preemptive key listening for GTK+
  *
@@ -342,20 +537,18 @@ spi_atk_register_event_listeners (void)
 }
 
 static void
-deregister_application (BonoboObject *app)
+deregister_application (SpiAppData *app)
 {
-  Accessibility_Registry registry = spi_atk_bridge_get_registry ();    
-  Accessibility_Registry_deregisterApplication (registry, BONOBO_OBJREF (app), &ev);
-
-  registry = bonobo_object_release_unref (registry, &ev);
-  
-  app = bonobo_object_unref (app);
+  const char *registry = spi_atk_bridge_get_registry ();
+  bridge_threads_leave ();
+  // todo: deregister
+  bridge_threads_enter ();
 }
 
 static void
 spi_atk_bridge_exit_func (void)
 {
-  BonoboObject *app = (BonoboObject *) this_app;
+  SpiAppData *app = (SpiAppData *) this_app;
 
   DBG (1, g_message ("exiting bridge\n"));
 
@@ -363,28 +556,32 @@ spi_atk_bridge_exit_func (void)
     {
       return;
     }
-  this_app = NULL;
+  if (atk_bridge_pid != getpid ())
+    {
+      _exit (0);
+    }
 
+  during_init_shutdown = TRUE;
+  exiting = TRUE;
+  /*
+   * Check whether we still have windows which have not been deleted.
+   */
+  spi_atk_tidy_windows ();
   /*
    *  FIXME: this may be incorrect for apps that do their own bonobo
    *  shutdown, until we can explicitly shutdown to get the ordering
    *  right.
    */
-  if (!bonobo_is_initialized ())
-    {
-      DBG (1, g_warning ("Re-initializing bonobo\n"));
-      g_assert (bonobo_init (0, NULL));
-      g_assert (bonobo_activate ());
-    }
-  
-  deregister_application (app);
-
+  if (!registry_died)
+    deregister_application (this_app);
+  this_app = NULL;
   DBG (1, g_message ("bridge exit func complete.\n"));
 
   if (g_getenv ("AT_BRIDGE_SHUTDOWN"))
     {
-      g_assert (!bonobo_debug_shutdown ());
     }
+  if (bridge_display)
+    XCloseDisplay (bridge_display);
 }
 
 void
@@ -392,13 +589,15 @@ gnome_accessibility_module_init (void)
 {
   atk_bridge_init (NULL, NULL);
 
-  g_print("Atk Accessibilty bridge initialized\n");
+  if (g_getenv ("AT_BRIDGE_SHUTDOWN"))
+    {
+       g_print("Atk Accessibility bridge initialized\n");
+    }
 }
 
 void
 gnome_accessibility_module_shutdown (void)
 {
-  BonoboObject *app = (BonoboObject *) this_app;
   int     i;
   GArray *ids = listener_ids;
   
@@ -406,115 +605,119 @@ gnome_accessibility_module_shutdown (void)
     {
       return;
     }
+  during_init_shutdown = TRUE;
   atk_bridge_initialized = FALSE;
-  this_app = NULL;
 
-  g_print("Atk Accessibilty bridge shutdown\n");
+  if (g_getenv ("AT_BRIDGE_SHUTDOWN"))
+    {
+       g_print("Atk Accessibility bridge shutdown\n");
+    }
 
   listener_ids = NULL;
-  atk_remove_focus_tracker (atk_bridge_focus_tracker_id);
+  if (atk_bridge_focus_tracker_id)
+        atk_remove_focus_tracker (atk_bridge_focus_tracker_id);
   
   for (i = 0; ids && i < ids->len; i++)
-  {
+    {
           atk_remove_global_event_listener (g_array_index (ids, guint, i));
-  }
+    }
   
-  atk_remove_key_event_listener (atk_bridge_key_event_listener_id);
+  if (atk_bridge_key_event_listener_id)
+          atk_remove_key_event_listener (atk_bridge_key_event_listener_id);
 
-  deregister_application (app);
+  deregister_application (this_app);
+  this_app = NULL;
+
+  misc = NULL;
+}
+
+static void emit(AtkObject *object, const char *name, int first_arg_type, ...)
+{
+  va_list args;
+  DBusMessage *sig;
+  char *path = spi_dbus_get_path(object);
+
+  spi_dbus_update_cache(&this_app->droute);
+  sig = dbus_message_new_signal(path, "org.freedesktop.atspi.Accessible", name);
+  va_start(args, first_arg_type);
+  if (first_arg_type != DBUS_TYPE_INVALID)
+  {
+    dbus_message_append_args_valist(sig, first_arg_type, args);
+  }
+  va_end(args);
+  dbus_connection_send(this_app->droute.bus, sig, NULL);
+  g_free(path);
+  dbus_message_unref(sig);
 }
 
 static void
 spi_atk_bridge_focus_tracker (AtkObject *object)
 {
-  SpiAccessible *source;
-  Accessibility_Event e;
-
-  source = spi_accessible_new (object);
-
-  e.type = "focus:";
-  e.source = BONOBO_OBJREF (source);
-  e.detail1 = 0;
-  e.detail2 = 0;
-  spi_init_any_nil (&e.any_data);
-  Accessibility_Registry_notifyEvent (spi_atk_bridge_get_registry (), &e, &ev);
-  if (BONOBO_EX (&ev)) registry_died = TRUE;
-  
-  Accessibility_Accessible_unref (e.source, &ev);
-  
-  CORBA_exception_free (&ev);
+  emit(object, "focus", DBUS_TYPE_INVALID);
 }
 
-static void
-spi_atk_emit_eventv (const GObject         *gobject,
-                    long                   detail1,
-                    long                   detail2,
-                    CORBA_any             *any,
-                    const char            *format, ...)
-{
-  va_list             args;
-  Accessibility_Event e;
-  SpiAccessible      *source;
-  AtkObject          *aobject;
-#ifdef SPI_BRIDGE_DEBUG
-  CORBA_string s;
-#endif
-  
-  va_start (args, format);
-  
-  if (ATK_IS_IMPLEMENTOR (gobject))
-    {
-      aobject = atk_implementor_ref_accessible (ATK_IMPLEMENTOR (gobject));
-      source  = spi_accessible_new (aobject);
-      g_object_unref (G_OBJECT (aobject));
-    }
-  else if (ATK_IS_OBJECT (gobject))
-    {
-      aobject = ATK_OBJECT (gobject);
-      source  = spi_accessible_new (aobject);
-    }
+static void emit_property_change(AtkObject *object, const char *name, int type, void *val)
+{
+  DBusMessage *sig;
+  char *path = spi_dbus_get_path(object);
+  DBusMessageIter iter, sub;
+  const char *type_as_string = NULL;
+
+  spi_dbus_update_cache(&this_app->droute);
+  if (type == DBUS_TYPE_OBJECT_PATH)
+  {
+    type_as_string = "o";
+    if (!val) val = "";
+  }
+  else if (type == DBUS_TYPE_STRING) type_as_string = "s";
+  else if (type == DBUS_TYPE_INT32) type_as_string = "i";
+  else if (type == DBUS_TYPE_UINT32) type_as_string = "o";
   else
-    {
-      aobject = NULL;
-      source  = NULL;
-      DBG (0, g_warning ("received property-change event from non-AtkImplementor"));
-    }
+  {
+    g_warning("Unknown type %d in property change signal", type);
+  }
+  sig = dbus_message_new_signal(path, "org.freedesktop.atspi.Accessible", "PropertyChanged");
+  dbus_message_iter_init_append(sig, &iter);
+  dbus_message_iter_append_basic(&iter, DBUS_TYPE_STRING, &name);
+  dbus_message_iter_open_container(&iter, DBUS_TYPE_VARIANT, type_as_string, &sub);
+  dbus_message_iter_append_basic(&sub, type, val);
+  dbus_message_iter_close_container(&iter, &sub);
+  dbus_connection_send(this_app->droute.bus, sig, NULL);
+  g_free(path);
+  dbus_message_unref(sig);
+}
 
-  if (source) 
+static void emit_rect(AtkObject *object, const char *name, AtkRectangle *rect)
+{
+  DBusMessage *sig;
+  char *path = spi_dbus_get_path(object);
+  DBusMessageIter iter, sub;
+  dbus_uint32_t x, y, width, height;
+
+  spi_dbus_update_cache(&this_app->droute);
+  x = rect->x;
+  y = rect->y;
+  width = rect->width;
+  height = rect->height;
+  sig = dbus_message_new_signal(path, "org.freedesktop.atspi.Accessible", "PropertyChanged");
+  if (sig)
     {
-      e.type = g_strdup_vprintf (format, args);
-      e.source = BONOBO_OBJREF (source);
-      e.detail1 = detail1;
-      e.detail2 = detail2;
-      if (any) e.any_data = *any;
-      else spi_init_any_nil (&e.any_data);
-      
-#ifdef SPI_BRIDGE_DEBUG
-      s = Accessibility_Accessible__get_name (BONOBO_OBJREF (source), &ev);
-      g_warning ("Emitting event '%s' (%lu, %lu) on %s",
-                e.type, e.detail1, e.detail2, s);
-      CORBA_free (s);
-#endif
-      CORBA_exception_init (&ev);
-      Accessibility_Registry_notifyEvent (spi_atk_bridge_get_registry (), 
-                                         &e, &ev);
-#ifdef SPI_BRIDGE_DEBUG
-      if (ev._major != CORBA_NO_EXCEPTION)
-       g_warning ("error emitting event %s, (%d) %s",
-                  e.type,
-                  ev._major,
-                  CORBA_exception_id(&ev));
-#endif       
-      if (BONOBO_EX (&ev)) registry_died = TRUE;
-      Accessibility_Accessible_unref (e.source, &ev);
-      
-      CORBA_exception_free (&ev);
-      
-      g_free (e.type);
+      dbus_message_iter_init_append (sig, &iter);
+      dbus_message_iter_append_basic(&iter, DBUS_TYPE_STRING, &name);
+      if (!dbus_message_iter_open_container
+         (&iter, DBUS_TYPE_STRUCT, NULL, &sub))
+       goto oom;
+      dbus_message_iter_append_basic (&sub, DBUS_TYPE_INT32, &x);
+      dbus_message_iter_append_basic (&sub, DBUS_TYPE_INT32, &y);
+      dbus_message_iter_append_basic (&sub, DBUS_TYPE_INT32, &width);
+      dbus_message_iter_append_basic (&sub, DBUS_TYPE_INT32, &height);
+      if (!dbus_message_iter_close_container (&iter, &sub))
+       goto oom;
     }
-
-  va_end (args);
-
+  dbus_connection_send(this_app->droute.bus, sig, NULL);
+oom:
+  g_free(path);
+  dbus_message_unref(sig);
 }
 
 static gboolean
@@ -524,30 +727,91 @@ spi_atk_bridge_property_event_listener (GSignalInvocationHint *signal_hint,
                                        gpointer data)
 {
   AtkPropertyValues *values;
-  GObject *gobject;
+  AtkObject *obj;
+  const gchar *prop_name;
+  const gchar *sp = NULL;
+  AtkObject *ao;
+  char *s_ao = NULL;
+  gint i;
+  const gchar *name = NULL;
 
 #ifdef SPI_BRIDGE_DEBUG
   GSignalQuery signal_query;
-  const gchar *name;
+  const gchar *signame;
   const gchar *s, *s2;
   
   g_signal_query (signal_hint->signal_id, &signal_query);
-  name = signal_query.signal_name;
+  signame = signal_query.signal_name;
 
   s2 = g_type_name (G_OBJECT_TYPE (g_value_get_object (param_values + 0)));
   s = atk_object_get_name (ATK_OBJECT (g_value_get_object (param_values + 0)));
   values = (AtkPropertyValues*) g_value_get_pointer (param_values + 1);
   DBG (2, g_message ("Received (property) signal %s:%s:%s from object %s (gail %s)\n",
-          g_type_name (signal_query.itype), name, values->property_name, s, s2));
+          g_type_name (signal_query.itype), signame, values->property_name, s, s2));
   
 #endif
 
-  gobject = g_value_get_object (param_values + 0);
+  obj = g_value_get_object (param_values + 0);
+  name = atk_object_get_name (obj);
   values = (AtkPropertyValues*) g_value_get_pointer (param_values + 1);
 
-  spi_atk_emit_eventv (gobject, 0, 0, NULL,
-                      "object:property-change:%s", values->property_name);
-
+  prop_name = values->property_name;
+  if (strcmp (prop_name, "accessible-name") == 0)
+    {
+      spi_dbus_notify_change(obj, FALSE, &this_app->droute);
+    }
+  else if (strcmp (prop_name, "accessible-description") == 0)
+    {
+      spi_dbus_notify_change(obj, FALSE, &this_app->droute);
+    }
+  else if (strcmp (prop_name, "accessible-parent") == 0)
+    {
+      spi_dbus_notify_change(obj, FALSE, &this_app->droute);
+    }
+  else if (strcmp (prop_name, "accessible-table-summary") == 0)
+    {
+      ao = atk_table_get_summary (ATK_TABLE (obj));
+      s_ao = spi_dbus_get_path(ao);
+      emit_property_change(obj, prop_name, DBUS_TYPE_OBJECT_PATH, s_ao);
+    }
+  else if (strcmp (prop_name, "accessible-table-column-header") == 0)
+    {
+      i = g_value_get_int (&(values->new_value));
+      ao = atk_table_get_column_header (ATK_TABLE (obj), i);
+      s_ao = spi_dbus_get_path(ao);
+      emit_property_change(obj, prop_name, DBUS_TYPE_OBJECT_PATH, s_ao);
+    }
+  else if (strcmp (prop_name, "accessible-table-row-header") == 0)
+    {
+      i = g_value_get_int (&(values->new_value));
+      ao = atk_table_get_row_header (ATK_TABLE (obj), i);
+      s_ao = spi_dbus_get_path(ao);
+      emit_property_change(obj, prop_name, DBUS_TYPE_OBJECT_PATH, s_ao);
+    }
+  else if (strcmp (prop_name, "accessible-table-row-description") == 0)
+    {
+      i = g_value_get_int (&(values->new_value));
+      sp = atk_table_get_row_description (ATK_TABLE (obj), i);
+      emit_property_change(obj, prop_name, DBUS_TYPE_STRING, (void *)&sp);
+    }
+  else if (strcmp (prop_name, "accessible-table-column-description") == 0)
+    {
+      i = g_value_get_int (&(values->new_value));
+      sp = atk_table_get_column_description (ATK_TABLE(obj), i);
+      emit_property_change(obj, prop_name, DBUS_TYPE_STRING, (void *)&sp);
+    }
+  else if (strcmp (prop_name, "accessible-table-caption-object") == 0)
+    {
+      ao = atk_table_get_caption (ATK_TABLE(obj));
+      sp = atk_object_get_name (ao);
+      emit_property_change(obj, prop_name, DBUS_TYPE_STRING, (void *)&sp);
+    }
+  else
+    {
+      long v = 0;
+      emit_property_change(obj, prop_name, DBUS_TYPE_INT32, &v);
+    }
+  if (s_ao) g_free(s_ao);
   return TRUE;
 }
 
@@ -557,9 +821,8 @@ spi_atk_bridge_state_event_listener (GSignalInvocationHint *signal_hint,
                                     const GValue *param_values,
                                     gpointer data)
 {
-  GObject *gobject;
+  AtkObject *obj;
   gchar *property_name;
-  gchar *type;
   unsigned long detail1;
 #ifdef SPI_BRIDGE_DEBUG
   GSignalQuery signal_query;
@@ -571,22 +834,25 @@ spi_atk_bridge_state_event_listener (GSignalInvocationHint *signal_hint,
           g_type_name (signal_query.itype), name);
 #endif
 
-  gobject = g_value_get_object (param_values + 0);
+  obj = ATK_OBJECT(g_value_get_object (param_values + 0));
   property_name = g_strdup (g_value_get_string (param_values + 1));
+  /* Ignore defunct for now; we'll send a tree update to remove it when
+     the object goes away */
+  /* Also ignore state changes for objects not yet broadcast */
+  if ((property_name && !strcmp(property_name, "defunct")) ||
+      !spi_dbus_object_is_known(obj))
+  {
+    g_free(property_name);
+    return;
+  }
   detail1 = (g_value_get_boolean (param_values + 2))
     ? 1 : 0;
-  type = g_strdup_printf ("object:state-changed:%s", property_name);
-  spi_atk_emit_eventv (gobject, 
-                      detail1,
-                      0,
-                      NULL,
-                      type);
+  emit(obj, "StateChanged", DBUS_TYPE_STRING, &property_name, DBUS_TYPE_UINT32, &detail1, DBUS_TYPE_INVALID);
   g_free (property_name);
-  g_free (type);
   return TRUE;
 }
 
-
+#if 0
 static void
 spi_init_keystroke_from_atk_key_event (Accessibility_DeviceEvent  *keystroke,
                                       AtkKeyEventStruct          *event)
@@ -599,8 +865,8 @@ spi_init_keystroke_from_atk_key_event (Accessibility_DeviceEvent  *keystroke,
   else
 #endif
   if (!event)
-    {
-      g_print ("WARNING: NULL key event!");
+    { /* this doesn't really need translating */
+      g_print (_("WARNING: NULL key event reported."));
     }
   
   keystroke->id        = (CORBA_long) event->keyval;
@@ -609,8 +875,14 @@ spi_init_keystroke_from_atk_key_event (Accessibility_DeviceEvent  *keystroke,
   keystroke->modifiers = (CORBA_unsigned_short) (event->state & 0xFFFF);
   if (event->string)
     {
+      gunichar c;
+
       keystroke->event_string = CORBA_string_dup (event->string);
-      keystroke->is_text = CORBA_TRUE;
+      c = g_utf8_get_char_validated (event->string, -1);
+      if (c > 0 && g_unichar_isprint (c))
+        keystroke->is_text = CORBA_TRUE;
+      else
+        keystroke->is_text = CORBA_FALSE;
     }
   else
     {
@@ -636,109 +908,52 @@ spi_init_keystroke_from_atk_key_event (Accessibility_DeviceEvent  *keystroke,
           keystroke->event_string, (int) keystroke->is_text, (unsigned long) keystroke->timestamp);
 #endif
 }
+#endif
 
 static gint
 spi_atk_bridge_key_listener (AtkKeyEventStruct *event, gpointer data)
 {
-  CORBA_boolean             result;
+  gboolean             result;
+#if 0
   Accessibility_DeviceEvent key_event;
-  Accessibility_DeviceEventController controller;
-       
-  if (BONOBO_EX (&ev))
-       g_warning ("failure: pre-listener get dec\n");
 
-  controller =
-    Accessibility_Registry_getDeviceEventController (
-           spi_atk_bridge_get_registry (), &ev);
+  CORBA_exception_init (&ev);
 
-  if (BONOBO_EX (&ev))
-    {
-      g_warning ("failure: no deviceeventcontroller found\n");
-      CORBA_exception_free (&ev);
-      registry_died = TRUE;
-      result = FALSE;
-    }
-  else
-    {
+  spi_init_keystroke_from_atk_key_event (&key_event, event);
 
-      spi_init_keystroke_from_atk_key_event (&key_event, event);
+  bridge_threads_leave ();
+  result = Accessibility_DeviceEventController_notifyListenersSync (
+         spi_atk_bridget_get_dec (), &key_event, &ev);
+  bridge_threads_enter ();
 
-      result = Accessibility_DeviceEventController_notifyListenersSync (
-        controller, &key_event, &ev);
+  if (key_event.event_string) CORBA_free (key_event.event_string);
 
-      bonobo_object_release_unref (controller, &ev);
+  if (BONOBO_EX(&ev))
+    {
+      result = FALSE;
       CORBA_exception_free (&ev);
     }
 
-  return result;
-}
-
-
-static void
-spi_atk_signal_emit_event (const GObject *gobject, 
-                          const GSignalQuery *signal_query,
-                          long detail1, 
-                          long detail2,
-                          const gchar *name, 
-                          const gchar *detail)
-{
-  CORBA_any any;
-  CORBA_char *sp = NULL;
-  AtkObject *ao;
-
-  if (signal_query->signal_id == atk_signal_text_changed)
-    {
-      sp = atk_text_get_text (ATK_TEXT (gobject),
-                             detail1,
-                             detail1+detail2);
-      spi_init_any_string (&any, &sp);
-    }
-#ifdef EXTENDED_OBJECT_EVENTS_ARE_WORKING
-  else if ((signal_query->signal_id == atk_signal_child_changed) && gobject)
-    {
-      ao = atk_object_ref_accessible_child (ATK_OBJECT (gobject), 
-                                           detail1);
-      if (ao) 
-       {
-         spi_init_any_object (&any, ao); 
-         atk_object_unref (ao);
-       }
-      else
-       {
-         spi_init_any_nil (&any);
-       }
-    }
 #endif
-  else
-    {
-      spi_init_any_nil (&any);
-    }
-
-  if (detail)
-    spi_atk_emit_eventv (gobject, detail1, detail2, &any,
-                        "object:%s:%s", name, detail);
-  else
-    spi_atk_emit_eventv (gobject, detail1, detail2, &any,
-                        "object:%s", name);
-  if (sp)
-    g_free (sp);
+  return result;
 }
 
-
-
 static gboolean
 spi_atk_bridge_signal_listener (GSignalInvocationHint *signal_hint,
                                guint n_param_values,
                                const GValue *param_values,
                                gpointer data)
 {
-  GObject *gobject;
+  AtkObject *obj;
   GSignalQuery signal_query;
   const gchar *name;
   const gchar *detail;
-  CORBA_char *sp;
-  
+  char *sp = NULL;
+  AtkObject *ao;
   gint detail1 = 0, detail2 = 0;
+  char *s_ao = NULL;
+  gchar *sig_name;
+  char *p;
 #ifdef SPI_BRIDGE_DEBUG
   const gchar *s, *s2;
 #endif 
@@ -747,9 +962,16 @@ spi_atk_bridge_signal_listener (GSignalInvocationHint *signal_hint,
 
   name = signal_query.signal_name;
   if (signal_hint->detail)
+  {
     detail = g_quark_to_string (signal_hint->detail);
+    sig_name = g_strdup_printf("object_%s_%s", name, detail);
+  }
   else
+  {
     detail = NULL;
+    sig_name = g_strdup_printf("object_%s", name);
+  }
+  while ((p = strchr(sig_name, '-')) != NULL) *p = '_';
 
 #ifdef SPI_BRIDGE_DEBUG
   s2 = g_type_name (G_OBJECT_TYPE (g_value_get_object (param_values + 0)));
@@ -759,15 +981,78 @@ spi_atk_bridge_signal_listener (GSignalInvocationHint *signal_hint,
           detail ? detail : "<NULL>", s ? s : "<NULL>" , s2);
 #endif
   
-  gobject = g_value_get_object (param_values + 0);
-  if (G_VALUE_TYPE (param_values + 1) == G_TYPE_INT)
-    detail1 = g_value_get_int (param_values + 1);
-  if (G_VALUE_TYPE (param_values + 2) == G_TYPE_INT)
-    detail2 = g_value_get_int (param_values + 2);
-  
-  spi_atk_signal_emit_event (gobject, &signal_query,
-                            detail2, detail2,
-                            name, detail);
+  obj = ATK_OBJECT(g_value_get_object (param_values + 0));
+
+  if (signal_query.signal_id == atk_signal_active_descendant_changed)
+    {
+      gpointer child = g_value_get_pointer (param_values + 1);
+
+      g_return_val_if_fail (ATK_IS_OBJECT (child), TRUE);
+
+      ao = ATK_OBJECT (child);
+
+      detail1 = atk_object_get_index_in_parent (ao);
+      s_ao = spi_dbus_get_path(child);
+      emit(obj, "name", DBUS_TYPE_OBJECT_PATH, s_ao, DBUS_TYPE_UINT32, &detail1, DBUS_TYPE_INVALID);
+      g_free(s_ao);
+    }
+  else if (signal_query.signal_id == atk_signal_link_selected)
+    {
+      if (G_VALUE_TYPE (param_values + 1) == G_TYPE_INT)
+        detail1 = g_value_get_int (param_values + 1);
+      emit(obj, "LinkSelected", DBUS_TYPE_UINT32, &detail1, DBUS_TYPE_INVALID);
+    }
+  else if (signal_query.signal_id == atk_signal_bounds_changed)
+    {
+      AtkRectangle *atk_rect = NULL;
+
+      if (G_VALUE_HOLDS_BOXED (param_values + 1))
+         atk_rect = g_value_get_boxed (param_values + 1);
+      emit_rect(obj, "BoundsChanged", atk_rect);
+    }
+  else if ((signal_query.signal_id == atk_signal_children_changed) && obj)
+    {
+      spi_dbus_notify_change(obj, FALSE, &this_app->droute);
+    }
+  else
+    {
+      if (n_param_values >= 2)
+        {
+          if (G_VALUE_TYPE (param_values + 1) == G_TYPE_INT)
+            detail1 = g_value_get_int (param_values + 1);
+          if (n_param_values >= 3)
+            {
+              if (G_VALUE_TYPE (param_values + 2) == G_TYPE_INT)
+                detail2 = g_value_get_int (param_values + 2);
+            }
+        }
+
+      if (signal_query.signal_id == atk_signal_text_changed)
+        {
+          sp = atk_text_get_text (ATK_TEXT (obj),
+                                 detail1,
+                                 detail1+detail2);
+          emit(obj, sig_name, DBUS_TYPE_UINT32, &detail1, DBUS_TYPE_UINT32, &detail2, DBUS_TYPE_STRING, &sp, DBUS_TYPE_INVALID);
+        }
+      else if (signal_query.signal_id == atk_signal_text_selection_changed)
+        {
+          /* Return NULL as the selected string */
+          // TODO
+          emit(obj, sig_name, DBUS_TYPE_INVALID);
+        }
+      else
+        {
+          emit(obj, sig_name, DBUS_TYPE_INVALID);
+        }
+    }
+
+  if (sp) 
+    g_free (sp);
+
+  if (s_ao)
+     g_free (s_ao);
+  g_free(sig_name);
+
   return TRUE;
 }
 
@@ -777,9 +1062,8 @@ spi_atk_bridge_window_event_listener (GSignalInvocationHint *signal_hint,
                                      const GValue *param_values,
                                      gpointer data)
 {
-  GObject *gobject;
+  AtkObject *obj;
   GSignalQuery signal_query;
-  CORBA_any any;
   const gchar *name, *s;
 #ifdef SPI_BRIDGE_DEBUG
   const gchar *s2;
@@ -788,6 +1072,7 @@ spi_atk_bridge_window_event_listener (GSignalInvocationHint *signal_hint,
   g_signal_query (signal_hint->signal_id, &signal_query);
 
   name = signal_query.signal_name;
+  gchar *sig_name;
 
 #ifdef SPI_BRIDGE_DEBUG
   s2 = g_type_name (G_OBJECT_TYPE (g_value_get_object (param_values + 0)));
@@ -796,12 +1081,84 @@ spi_atk_bridge_window_event_listener (GSignalInvocationHint *signal_hint,
           g_type_name (signal_query.itype), name, s ? s : "<NULL>" , s2);
 #endif
   
-  gobject = g_value_get_object (param_values + 0);
+  obj = ATK_OBJECT(g_value_get_object (param_values + 0));
 
-  s = atk_object_get_name (ATK_OBJECT (gobject));
-  spi_init_any_string (&any, &s);
-  
-  spi_atk_emit_eventv (gobject, 0, 0, &any,
-                      "window:%s", name);
+  s = atk_object_get_name (obj);
+  sig_name = g_strdup_printf("window_%s", name);
+  emit(obj, sig_name, DBUS_TYPE_STRING, &s, DBUS_TYPE_INVALID);
+  g_free(sig_name);
+
+  return TRUE;
+}
+
+static gboolean
+spi_atk_bridge_document_event_listener (GSignalInvocationHint *signal_hint,
+                                      guint n_param_values,
+                                      const GValue *param_values,
+                                      gpointer data)
+{
+  AtkObject *obj;
+  GSignalQuery signal_query;
+  const gchar *name, *s;
+  gchar *sig_name;
+#ifdef SPI_BRIDGE_DEBUG
+  const gchar *s2;
+#endif
+
+  g_signal_query (signal_hint->signal_id, &signal_query);
+
+  name = signal_query.signal_name;
+
+#ifdef SPI_BRIDGE_DEBUG
+  s2 = g_type_name (G_OBJECT_TYPE (g_value_get_object (param_values + 0)));
+  s = atk_object_get_name (ATK_OBJECT (g_value_get_object (param_values + 0)));
+  fprintf (stderr, "Received signal %s:%s from object %s (gail %s)\n",
+           g_type_name (signal_query.itype), name, s ? s : "<NULL>" , s2);
+#endif
+
+  obj = ATK_OBJECT(g_value_get_object (param_values + 0));
+
+  s = atk_object_get_name (obj);
+  sig_name = g_strdup_printf("document_%s", name);
+  emit(obj, sig_name, DBUS_TYPE_STRING, &s, DBUS_TYPE_INVALID);
+  g_free(sig_name);
   return TRUE;
 }
+
+static void
+spi_atk_tidy_windows (void)
+{
+  AtkObject *root;
+  gint n_children;
+  gint i;
+
+  root = atk_get_root ();
+  n_children = atk_object_get_n_accessible_children (root);
+  for (i = 0; i < n_children; i++)
+    {
+      AtkObject *child;
+      AtkStateSet *stateset;
+      const gchar *name;
+     
+      child = atk_object_ref_accessible_child (root, i);
+      stateset = atk_object_ref_state_set (child);
+      
+      name = atk_object_get_name (child);
+      if (atk_state_set_contains_state (stateset, ATK_STATE_ACTIVE))
+        {
+         emit(child, "window:deactivate", DBUS_TYPE_STRING, &name, DBUS_TYPE_INVALID);
+        }
+      g_object_unref (stateset);
+
+      emit(child, "window:destroy", DBUS_TYPE_STRING, &name, DBUS_TYPE_INVALID);
+      g_object_unref (child);
+    }
+}
+
+static void
+reinit_register_vars (void)
+{
+  registry = NULL;
+  device_event_controller = NULL;
+  this_app = NULL;
+}