GNOME Bugzilla – Bug 598289
Add "window-created" signal to MetaDisplay, "unmanaged" signal for MetaWindow
Last modified: 2009-10-14 18:39:26 UTC
For some consumers it's significantly more convenient to be able to directly connect to a signal on the Window to know when Mutter is done with it, rather than having to connect to each Workspace object (and handle workspace additions, etc.). Similarly, add window-created which acts globally.
Created attachment 145357 [details] [review] Add "window-created" signal to MetaDisplay, "unmanaged" signal for MetaWindow
Unless I am misreading the patch, the window-created signal does not seem to be ever emitted. Also, I don't know if the ::unmanaged signal is of much use, since by the time it's emitted there is very little you can do with the MetaWindow object, it's not far removed from just having a weak ref on the MetaWindow object.
(In reply to comment #2) > Unless I am misreading the patch, the window-created signal does not seem to be > ever emitted. Oops yes, will fix. > Also, I don't know if the ::unmanaged signal is of much use, > since by the time it's emitted there is very little you can do with the > MetaWindow object, it's not far removed from just having a weak ref on the > MetaWindow object. Well, it's tricky. I do have some C code I could convert to use weakrefs, but in the presence of garbage collection, we really want to be able to have a more deterministic lifecycle for "heavyweight" objects like MetaWindow and Clutter actors. ClutterActor has the DESTROY signal which works fine for those, it'd be nice to have something similar for MetaWindow.
(In reply to comment #3) > Well, it's tricky. I do have some C code I could convert to use weakrefs, but > in the presence of garbage collection, we really want to be able to have a more > deterministic lifecycle for "heavyweight" objects like MetaWindow and Clutter > actors. ClutterActor has the DESTROY signal which works fine for those, it'd > be nice to have something similar for MetaWindow. OK, that makes sense. That reminds me, we need to start adding doc comments to signals, I don't think any of our signals are documented :(.
Attachment 145357 [details] pushed as 9311add - Add "window-created" signal to MetaDisplay, "unmanaged" signal for MetaWindow