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 656306 - Cally should use AtkWindow and remove that hack on CallyUtil
Cally should use AtkWindow and remove that hack on CallyUtil
Status: RESOLVED FIXED
Product: clutter
Classification: Platform
Component: cally
unspecified
Other Linux
: Normal normal
: ---
Assigned To: clutter-maint
clutter-maint
Depends on: 638924
Blocks:
 
 
Reported: 2011-08-10 18:42 UTC by Alejandro Piñeiro Iglesias (IRC: infapi00)
Modified: 2011-08-30 10:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
CallyStage implementing AtkWindow (5.80 KB, patch)
2011-08-10 18:43 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
none Details | Review
We just allow to register atk events now (2.47 KB, patch)
2011-08-10 18:44 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
none Details | Review
Now the window events registration should be done asking for the AtkWindow interface (1.56 KB, patch)
2011-08-10 18:44 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
none Details | Review
Test doesn't compile (1.81 KB, patch)
2011-08-10 18:50 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
committed Details | Review
Using AtkWindow on cally stage (6.32 KB, patch)
2011-08-24 16:00 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
committed Details | Review
We don't need to explicitly manage window events (2.47 KB, patch)
2011-08-24 16:00 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
committed Details | Review
Updating a11y tests with the proper window format (1.55 KB, patch)
2011-08-24 16:01 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
committed Details | Review

Description Alejandro Piñeiro Iglesias (IRC: infapi00) 2011-08-10 18:42:30 UTC
At this moment when you register a global event listener you can register for Atk events and other kind of events not specified. At this moment, Atk and window events.

As this is madness, we decided to allow to just register to Atk events (bug 649577), and in order to do that, implement AtkWindow (638924).

This bug is just the migration to that new interface.
Comment 1 Alejandro Piñeiro Iglesias (IRC: infapi00) 2011-08-10 18:43:47 UTC
Created attachment 193586 [details] [review]
CallyStage implementing AtkWindow
Comment 2 Alejandro Piñeiro Iglesias (IRC: infapi00) 2011-08-10 18:44:12 UTC
Created attachment 193588 [details] [review]
We just allow to register atk events now
Comment 3 Alejandro Piñeiro Iglesias (IRC: infapi00) 2011-08-10 18:44:49 UTC
Created attachment 193589 [details] [review]
Now the window events registration should be done asking for the AtkWindow interface
Comment 4 Alejandro Piñeiro Iglesias (IRC: infapi00) 2011-08-10 18:50:09 UTC
Created attachment 193590 [details] [review]
Test doesn't compile

This is weird.

I needed to make this change because if not, when using an atk with the patches at 638924 I get the error:

make[3]: Entering directory `/home/devel/GNOME3/source/clutter/tests/micro-bench'
  CCLD   test-text
../../clutter/.libs/libclutter-glx-1.0.so: undefined reference to `atk_window_get_type'
 
And it is weird because this doesn't happens with other ATK symbols, and as far as I see I'm not doing anything estrange with that atkwindow.

But in the same way, not all the tests are failing to compile. So I modified micro-bench and performance tests Makefile.am in order to be more similar to the working ones (LDADD and so on). And in that way seems to work.

Further investigation required.
Comment 5 Alejandro Piñeiro Iglesias (IRC: infapi00) 2011-08-24 16:00:30 UTC
Created attachment 194632 [details] [review]
Using AtkWindow on cally stage
Comment 6 Alejandro Piñeiro Iglesias (IRC: infapi00) 2011-08-24 16:00:52 UTC
Created attachment 194633 [details] [review]
We don't need to explicitly manage window events
Comment 7 Alejandro Piñeiro Iglesias (IRC: infapi00) 2011-08-24 16:01:22 UTC
Created attachment 194634 [details] [review]
Updating a11y tests with the proper window format