cally: Refactoring "window:create" and "window:destroy" emission code
authorAlejandro Piñeiro <apinheiro@igalia.com>
Tue, 6 Jul 2010 11:25:44 +0000 (13:25 +0200)
committerAlejandro Piñeiro <apinheiro@igalia.com>
Wed, 7 Jul 2010 13:47:56 +0000 (15:47 +0200)
commit137790bec9d4c29a4e5504ea49623884e90e3cca
treeb73e79e17d1ce3a5bb928821d262f980f1724f42
parentffd1f125604a2a25015804bcf2b8ebb6503fdd77
cally: Refactoring "window:create" and "window:destroy" emission code

Previously "window:create" and "window:destroy" were emitted on
CallyUtil. Although it works, and CallyUtil already have callbacks to
stage_added/removed signals, I think that it is more tidy/clear to do
that on CallyRoot:

  * CallyRoot already has code to manage ClutterStage addition/removal

  * In fact, we can see CallyRoot as the object exposing the a11y
    information from ClutterStageManager, so it fits better here.

  * CallyUtil callbacks these signals are related to key event
    listeners (key snooper simulation). One of the main CallyUtil
    responsabilities is managing event (connecting, emitting), so I
    would prefer to not start to add/mix more functionalities here.

Ideally it would be better to emit all CallyStage methods from
CallyStage, but it is clear that "create" and "destroy" are more easy
to emit from a external object
clutter/cally/cally-root.c
clutter/cally/cally-util.c