| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
|
|\
| |
| |
| |
| | |
_atspi_dbus_call_partial: add missing va_end call
See merge request GNOME/at-spi2-core!73
|
| |
| |
| |
| |
| | |
The C standard says that va_end has to be called in the same function as
va_start.
|
|/
|
|
|
| |
We shouldn't call _atspi_device_listener_get_path before checking that
listener is not NULL.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
processing the event
Once we demarshal, the event.source should not be NULL. It can come
from the message path if it is *not* being a ScreenReader event, or
from a variant argument.
Fixes static-scan warnings of this sort:
../../../atspi/atspi-event-listener.c:248:9: warning: Access to field 'accessible_parent' results in a dereference of a null pointer (loaded from field 'source') [core.NullDereference]
if (event->source->accessible_parent)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
|
|
|
|
|
|
|
|
|
|
|
| |
is NULL
I don't know why the interface would be NULL, but
dbus_message_get_interface() can return that.
We return DBUS_HANDLER_RESULT_NOT_YET_HANDLED in the same way as the
other error case, but note that the caller of this function ignores
the return value.
|
|
|
|
| |
To avoid using a freed pointer. It's harmless, but static-scan complains.
|
|
|
|
|
| |
This function is only called by get_accessibility_bus_address_x11(),
which is already conditional on HAVE_X11.
|
|
|
|
|
| |
device_remove_datum already implicitly casts its cb to a
AtspiDeviceListenerSimpleCB, which takes a const *event.
|
| |
|
|
|
|
| |
Fixes https://gitlab.gnome.org/GNOME/at-spi2-core/issues/35
|
|
|
|
|
|
|
|
| |
The parameter is marked with (transfer full), and the code expects the caller
to free the event, so adjust the prototype to clarify the expected behavior.
Also fix a related memory leak in atspi-device-legacy.c.
Fixes #31
|
| |
|
|
|
|
| |
This is likely what the backed-out part of !53 was trying to do.
|
|
|
|
| |
Fixes https://gitlab.gnome.org/GNOME/at-spi2-core/issues/30
|
|
|
|
|
|
|
|
|
|
|
| |
* Release memory before pointer overwrite.
* Reduce memory leaks reported in ref_accessible.
* Remove leak connected with atspi_state_type_get_type.
* Remove memory leak in
atspi_event_listener_register_from_callback_full.
* Reduce memory leaks reported on program exit.
Change-Id: Ifde65dd44643f2e58442cf70809703f979436e09
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
This is intended to replace the registry-based method for capturing
keystrokes. It is needed because gtk 4 no longer sends key notifications
in a way that atk-bridge can process them. Unlike the original API, key
grabs are separated from key notifications. Clients wishing to consume
keystrokes must proactively register a grab for the given key. Currently,
there is a backend for X11 and an unfinished legacy back end using the old
registry-based method. Hopefully, there will be a mutter/wayland back end in
the future, but we need to define a protocol there first.
|
|
|
|
|
|
|
|
| |
The g_object_run_dispose needs bus as below.
g_object_run_dispose > atspi_application_dispose > _atspi_bus
So close, unref and set bus to NULL after using bus is done.
|
| |
|
|
|
|
|
|
| |
We should not modify event_listeners while we are iterating through it.
Otherwise, we may crash. If an event listener is removed from within an event
callback, then defer the removal until the callback has ended.
|
|
|
|
| |
Fixes https://gitlab.gnome.org/GNOME/at-spi2-core/issues/24
|
| |
|
|\
| |
| |
| |
| | |
Keep indentation
See merge request GNOME/at-spi2-core!47
|
| | |
|
|/
|
|
| |
Thanks to Shimwoo Kim for spotting.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Atspi_accessible_set_cache_mask checks that it is being passed an
application's root accessible, and this sometimes fails and generates a
warning similar to the following, even when orca passes what it sees as the
application's root accessible:
(orca:18809): dbind-CRITICAL **: 09:27:25.951: atspi_accessible_set_cache_mask: assertion 'accessible == accessible->parent.app->root' failed
I suspect that this is caused by atk-bridge being initialized and caching
the root accessible and atk_get_root() is subsequently redefined, in the case
of an application that loads more than one atk implementor, such as Firefox.
This is indicative of a problem that should be fixed elsewhere--possibly
gtk+ and/or firefox shouldn't override atk_get_root if it already has an
implementation--but warning and failing here isn't helpful.
|
|
|
|
|
|
| |
Properly remove event listeners from the list when they are deregistered.
Fixes a crash that can happen when orca exits. Similar issue to
https://gitlab.gnome.org/GNOME/at-spi2-core/issues/22
|
| |
|
|\
| |
| |
| |
| | |
meson: De-duplicate deps of 'Requires' in pkgconfig file
See merge request GNOME/at-spi2-core!28
|
| |
| |
| |
| |
| |
| | |
Fix my mistake of listing dependencies in 'requires' as
a string instead of as pkgconfig dependencies. It was
causing duplicates in 'Requires.private'.
|
| |
| |
| |
| | |
Fixes #22.
|
| |
| |
| |
| | |
https://gitlab.gnome.org/GNOME/orca/issues/36
|
| |
| |
| |
| |
| |
| | |
Add a signal to allow a screen reader to indicate that its mode has changed.
https://gitlab.gnome.org/GNOME/orca/issues/36
|
|/
|
|
|
|
|
|
|
| |
For screen reader events, look at the name of the signal and convert it,
rather than hard-coding "RegionChanged." This is necessary to allow
screen_reader_signal_watcher to process signals that will be added in the
future.
https://gitlab.gnome.org/GNOME/orca/issues/36
|
| |
|
|
|
|
|
|
| |
These roles are needed to improve the accessibility of rich text editors
and other collaborative tools. The new roles parallel what was added to
IAccessible2 in Windows.
|
|
|
|
|
|
|
|
|
| |
The generated enum type files can be included in source packages meant
for debugging, and thus need to be reproducible. Replace the absolute
include of the header with the basename. This is sufficient because the
target include files are always in the include path anyway.
Signed-off-by: Joshua Watt <JPEWhacker@gmail.com>
|
| |
|
|
|
|
|
|
|
| |
Clarify that the length parameter expects a count of bytes. This matches
the corresponding atk documentation.
Fixes https://gitlab.gnome.org/GNOME/at-spi2-core/issues/9
|
|
|
|
| |
Fixes https://gitlab.gnome.org/GNOME/at-spi2-core/issues/2
|
| |
|
|\
| |
| |
| |
| | |
misc: Do not warn on no reply from pending get_items call
See merge request GNOME/at-spi2-core!22
|
| |
| |
| |
| |
| |
| |
| |
| | |
Like DBUS_ERROR_SERVICE_UNKNOWN, this may happen in normal use.
For instance, during the at-spi2-atk testsuite, in the
atk_test_accessible_set_cache_mask testcase, we may make the application
quit before get_items gets to be replied to.
|
| |
| |
| |
| |
| |
| | |
In case atspi_application_dispose was previously called for an
application, the app field of its root will be NULL. In such a case
cleanup should not try to dispose it.
|
|/
|
|
|
|
|
|
| |
The children array may contain NULL entries, when the children have not
all been inspected yet for instance. In such a case
atspi_accessible_dispose's call to g_ptr_array_free would warn out when
freeing array items with g_object_unref. We should thus check for NULL
to avoid such warning.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mutter now has a remote desktop interface with methods to synthesize keyboard
and mouse events. This functionality has not been available through AT-SPI on
wayland since it uses X-specific functions.
Also add atspi_set_reference_window.
Probably a work in progress; doesn't appear to be behaving as intended.
Also requires that introspection be enabled on gnome-shell. Ie,
gsettings set org.gnome.shell introspect true
https://bugzilla.gnome.org/show_bug.cgi?id=709999
https://gitlab.gnome.org/GNOME/at-spi2-core/issues/14
|