commit b8ab14fe046aecd8991f47f191f772974d5920f8
author Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com> Tue Mar 27 17:36:36 2012 +0300
committer Kristian Høgsberg <krh@bitplanet.net> Tue Mar 27 16:36:53 2012 -0400
tree fb64310fbf06c92fd172861af92e70e0e97d5a35
parent f9247ddc315bfbd7ec28867c5b91145ef1d90bf1

compositor: add a weston_surface_is_mapped() helper

src/compositor.c

diff --git a/src/compositor.c b/src/compositor.c
index c83731c..45c82c9 100644
--- a/src/compositor.c
+++ b/src/compositor.c
@@ -418,7 +418,7 @@
 	pixman_region32_union(&surface->damage, &surface->damage,
 			      &surface->transform.boundingbox);
 
-	if (surface->output)
+	if (weston_surface_is_mapped(surface))
 		weston_surface_assign_output(surface);
 
 	weston_compositor_schedule_repaint(surface->compositor);
@@ -530,6 +530,15 @@
 	surface->geometry.dirty = 1;
 }
 
+WL_EXPORT int
+weston_surface_is_mapped(struct weston_surface *surface)
+{
+	if (surface->output)
+		return 1;
+	else
+		return 0;
+}
+
 WL_EXPORT uint32_t
 weston_compositor_get_time(void)
 {
@@ -623,7 +632,7 @@
 			     struct weston_surface, surface.resource);
 	struct weston_compositor *compositor = surface->compositor;
 
-	if (surface->output)
+	if (weston_surface_is_mapped(surface))
 		weston_surface_unmap(surface);
 
 	if (surface->texture)
@@ -1119,7 +1128,7 @@
 	struct weston_shell *shell = es->compositor->shell;
 	struct wl_buffer *buffer;
 
-	if (!buffer_resource && !es->output)
+	if (!buffer_resource && !weston_surface_is_mapped(es))
 		return;
 
 	if (es->buffer) {
@@ -1127,7 +1136,7 @@
 		wl_list_remove(&es->buffer_destroy_listener.link);
 	}
 
-	if (!buffer_resource && es->output) {
+	if (!buffer_resource && weston_surface_is_mapped(es)) {
 		weston_surface_unmap(es);
 		es->buffer = NULL;
 		return;
@@ -1146,7 +1155,7 @@
 		pixman_region32_init(&es->opaque);
 	}
 
-	if (es->output == NULL) {
+	if (!weston_surface_is_mapped(es)) {
 		shell->map(shell, es, buffer->width, buffer->height, sx, sy);
 	} else if (es->force_configure || sx != 0 || sy != 0 ||
 		   es->geometry.width != buffer->width ||
@@ -1879,12 +1888,12 @@
 		wl_list_remove(&device->sprite->buffer_destroy_listener.link);
 
 	if (!buffer_resource) {
-		if (device->sprite->output)
+		if (weston_surface_is_mapped(device->sprite))
 			weston_surface_unmap(device->sprite);
 		return;
 	}
 
-	if (!device->sprite->output) {
+	if (!weston_surface_is_mapped(device->sprite)) {
 		wl_list_insert(&compositor->cursor_layer.surface_list,
 			       &device->sprite->layer_link);
 		weston_surface_assign_output(device->sprite);
@@ -2003,7 +2012,7 @@
 static void
 device_map_drag_surface(struct weston_input_device *device)
 {
-	if (device->drag_surface->output ||
+	if (weston_surface_is_mapped(device->drag_surface) ||
 	    !device->drag_surface->buffer)
 		return;