After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 119693 - AccessibleActiveDescendantChangedEvent_getActiveDescendant doesn't work
AccessibleActiveDescendantChangedEvent_getActiveDescendant doesn't work
Status: RESOLVED FIXED
Product: at-spi
Classification: Platform
Component: javabridge
0.0.1
Other Linux
: High major
: ---
Assigned To: padraig.obriain
padraig.obriain
Depends on:
Blocks: 117576
 
 
Reported: 2003-08-12 08:09 UTC by remus draica
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.3/2.4


Attachments
Proposed patch (1.54 KB, patch)
2003-08-13 13:54 UTC, padraig.obriain
none Details | Review

Description remus draica 2003-08-12 08:09:59 UTC
In case of event "object:active-descendant", when
AccessibleActiveDescendantChangedEvent_getActiveDescendant is called,
always a NULL pointer is obtained.

Apps to reproduce: 
   SwingSet2 JTreeDemo page-tab
or StarOfice Tools/Options (tree in left side of dialog).
Comment 1 padraig.obriain 2003-08-12 09:49:44 UTC
I can reproduce this problem.

First I would like to clarify what the name of the event should be.

GTK applications emit the event "object:active-descendant-changed". I
assume that that is what the name should be and the code in
at-spi/tests/event-listener-test.c which listens for
"object:active-descendant" is wrong.
Comment 2 remus draica 2003-08-12 13:25:09 UTC
Seems that JAB emits "object:active-descendant", not
"object:active-descendant-changed". For test I used a proper file.
So, even the event name is wrong in JAB.
Comment 3 padraig.obriain 2003-08-13 13:54:31 UTC
Created attachment 19181 [details] [review]
Proposed patch
Comment 4 remus draica 2003-08-14 06:22:03 UTC
Patch is fine. Please apply.
Comment 5 padraig.obriain 2003-08-14 07:30:42 UTC
Patch committed to CVS HEAD.