[PATCH weston] xwm: don't set inactive surfaces as top level

[Bill Spitzak]

On 01/12/2015 09:14 AM, Giulio Camuffo wrote:

> +static bool
> +weston_wm_window_inactive(struct weston_wm_window *window)
> +{
> +	struct weston_wm *wm = window->wm;
> +
> +	return window->type == wm->atom.net_wm_window_type_tooltip ||
> +	       window->type == wm->atom.net_wm_window_type_dropdown ||
> +	       window->type == wm->atom.net_wm_window_type_dnd ||
> +	       window->type == wm->atom.net_wm_window_type_combo ||
> +	       window->type == wm->atom.net_wm_window_type_popup;
> +}

I think maybe this should be called "window_type_inactive" because the 
current name implies this is some kind of check for whether the window 
is "currently inactive". Also is it necessary to namespace the static 
function with "weston_wm_" prefix?