window: Return -1 if meta_window_get_monitor is called on an unmanaged window

As opposed to crashing. In this case, letting the caller deal with
it is the best policy, since this is public API.

https://bugzilla.gnome.org/show_bug.cgi?id=788834
This commit is contained in:
Sam Spilsbury 2017-10-11 00:39:40 +08:00 committed by Marco Trevisan
parent 768ec15ea0
commit 8626c69c2f

View File

@ -3750,11 +3750,15 @@ maybe_move_attached_dialog (MetaWindow *window,
* *
* Gets index of the monitor that this window is on. * Gets index of the monitor that this window is on.
* *
* Return Value: The index of the monitor in the screens monitor list * Return Value: The index of the monitor in the screens monitor list, or -1
* if the window has been recently unmanaged and does not have a monitor.
*/ */
int int
meta_window_get_monitor (MetaWindow *window) meta_window_get_monitor (MetaWindow *window)
{ {
if (!window->monitor)
return -1;
return window->monitor->number; return window->monitor->number;
} }