compositor: Don't try and use a NULL sprite for the device pointer
[profile/ivi/weston.git] / compositor / compositor.c
index 71fab71..e0d7b32 100644 (file)
@@ -1,19 +1,24 @@
 /*
- * Copyright © 2008 Kristian Høgsberg
+ * Copyright © 2010-2011 Intel Corporation
+ * Copyright © 2008-2011 Kristian Høgsberg
  *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
+ * Permission to use, copy, modify, distribute, and sell this software and
+ * its documentation for any purpose is hereby granted without fee, provided
+ * that the above copyright notice appear in all copies and that both that
+ * copyright notice and this permission notice appear in supporting
+ * documentation, and that the name of the copyright holders not be used in
+ * advertising or publicity pertaining to distribution of the software
+ * without specific, written prior permission.  The copyright holders make
+ * no representations about the suitability of this software for any
+ * purpose.  It is provided "as is" without express or implied warranty.
  *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ * THE COPYRIGHT HOLDERS DISCLAIM ALL WARRANTIES WITH REGARD TO THIS
+ * SOFTWARE, INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND
+ * FITNESS, IN NO EVENT SHALL THE COPYRIGHT HOLDERS BE LIABLE FOR ANY
+ * SPECIAL, INDIRECT OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER
+ * RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF
+ * CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN
+ * CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
 #define _GNU_SOURCE
@@ -44,7 +49,6 @@
  * advertise that through a service on the session dbus.
  */
 static const char *option_socket_name = NULL;
-static const char *option_background = "background.png";
 static int option_idle_time = 300;
 
 static struct wl_list child_process_list;
@@ -204,6 +208,7 @@ surface_handle_buffer_destroy(struct wl_listener *listener,
                                               buffer_destroy_listener);
        struct wl_buffer *buffer = (struct wl_buffer *) resource;
 
+       wl_list_init(&es->buffer_destroy_listener.link);
        if (es->buffer == buffer)
                es->buffer = NULL;
 }
@@ -241,10 +246,10 @@ wlsc_surface_create(struct wlsc_compositor *compositor,
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);
 
-       surface->surface.client = NULL;
+       surface->surface.resource.client = NULL;
 
        surface->compositor = compositor;
-       surface->visual = NULL;
+       surface->visual = WLSC_NONE_VISUAL;
        surface->image = EGL_NO_IMAGE_KHR;
        surface->saved_texture = 0;
        surface->x = x;
@@ -252,7 +257,9 @@ wlsc_surface_create(struct wlsc_compositor *compositor,
        surface->width = width;
        surface->height = height;
 
+       surface->fullscreen_output = NULL;
        surface->buffer = NULL;
+       surface->output = NULL;
 
        pixman_region32_init(&surface->damage);
        pixman_region32_init(&surface->opaque);
@@ -307,8 +314,6 @@ WL_EXPORT void
 wlsc_surface_configure(struct wlsc_surface *surface,
                       int x, int y, int width, int height)
 {
-       struct wlsc_compositor *compositor = surface->compositor;
-
        wlsc_surface_damage_below(surface);
 
        surface->x = x;
@@ -320,7 +325,7 @@ wlsc_surface_configure(struct wlsc_surface *surface,
        wlsc_surface_damage(surface);
 
        pixman_region32_fini(&surface->opaque);
-       if (surface->visual == &compositor->compositor.rgb_visual)
+       if (surface->visual == WLSC_RGB_VISUAL)
                pixman_region32_init_rect(&surface->opaque,
                                          surface->x, surface->y,
                                          surface->width, surface->height);
@@ -339,7 +344,29 @@ wlsc_compositor_get_time(void)
 }
 
 static void
-destroy_surface(struct wl_resource *resource, struct wl_client *client)
+wlsc_compositor_repick(struct wlsc_compositor *compositor)
+{
+       struct wlsc_input_device *device;
+       struct wlsc_surface *surface;
+       int32_t sx, sy;
+       uint32_t time;
+
+       time = wlsc_compositor_get_time();
+       wl_list_for_each(device, &compositor->input_device_list, link) {
+               if (device->input_device.grab)
+                       continue;
+               surface = pick_surface(&device->input_device, &sx, &sy);
+               wl_input_device_set_pointer_focus(&device->input_device,
+                                                 &surface->surface,
+                                                 time,
+                                                 device->input_device.x,
+                                                 device->input_device.y,
+                                                 sx, sy);
+       }
+}
+
+static void
+destroy_surface(struct wl_resource *resource)
 {
        struct wlsc_surface *surface =
                container_of(resource, struct wlsc_surface, surface.resource);
@@ -348,6 +375,8 @@ destroy_surface(struct wl_resource *resource, struct wl_client *client)
        wlsc_surface_damage_below(surface);
 
        wl_list_remove(&surface->link);
+       wlsc_compositor_repick(compositor);
+
        if (surface->saved_texture == 0)
                glDeleteTextures(1, &surface->texture);
        else
@@ -387,7 +416,18 @@ wlsc_buffer_attach(struct wl_buffer *buffer, struct wl_surface *surface)
                             es->pitch, buffer->height, 0,
                             GL_BGRA_EXT, GL_UNSIGNED_BYTE,
                             wl_shm_buffer_get_data(buffer));
-               es->visual = buffer->visual;
+
+               switch (wl_shm_buffer_get_format(buffer)) {
+               case WL_SHM_FORMAT_ARGB32:
+                       es->visual = WLSC_ARGB_VISUAL;
+                       break;
+               case WL_SHM_FORMAT_PREMULTIPLIED_ARGB32:
+                       es->visual = WLSC_PREMUL_ARGB_VISUAL;
+                       break;
+               case WL_SHM_FORMAT_XRGB32:
+                       es->visual = WLSC_RGB_VISUAL;
+                       break;
+               }
 
                surfaces_attached_to = buffer->user_data;
 
@@ -401,7 +441,9 @@ wlsc_buffer_attach(struct wl_buffer *buffer, struct wl_surface *surface)
                                             buffer, NULL);
                
                ec->image_target_texture_2d(GL_TEXTURE_2D, es->image);
-               es->visual = buffer->visual;
+
+               /* FIXME: we need to get the visual from the wl_buffer */
+               es->visual = WLSC_PREMUL_ARGB_VISUAL;
                es->pitch = es->width;
        }
 }
@@ -452,7 +494,7 @@ create_sprite_from_png(struct wlsc_compositor *ec,
                return NULL;
        }
 
-       sprite->visual = &ec->compositor.premultiplied_argb_visual;
+       sprite->visual = WLSC_PREMUL_ARGB_VISUAL;
        sprite->width = width;
        sprite->height = height;
        sprite->image = EGL_NO_IMAGE_KHR;
@@ -513,30 +555,6 @@ create_pointer_images(struct wlsc_compositor *ec)
        }
 }
 
-static struct wlsc_surface *
-background_create(struct wlsc_output *output, const char *filename)
-{
-       struct wlsc_surface *background;
-       struct wlsc_sprite *sprite;
-
-       background = wlsc_surface_create(output->compositor,
-                                        output->x, output->y,
-                                        output->current->width,
-                                        output->current->height);
-       if (background == NULL)
-               return NULL;
-
-       sprite = create_sprite_from_png(output->compositor, filename, 0);
-       if (sprite == NULL) {
-               free(background);
-               return NULL;
-       }
-
-       wlsc_sprite_attach(sprite, &background->surface);
-
-       return background;
-}
-
 static int
 texture_region(struct wlsc_surface *es, pixman_region32_t *region)
 {
@@ -645,14 +663,21 @@ wlsc_surface_draw(struct wlsc_surface *es,
        if (!pixman_region32_not_empty(&repaint))
                return;
 
-       if (es->visual == &ec->compositor.argb_visual) {
+       switch (es->visual) {
+       case WLSC_ARGB_VISUAL:
                glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);
                glEnable(GL_BLEND);
-       } else if (es->visual == &ec->compositor.premultiplied_argb_visual) {
+               break;
+       case WLSC_PREMUL_ARGB_VISUAL:
                glBlendFunc(GL_ONE, GL_ONE_MINUS_SRC_ALPHA);
                glEnable(GL_BLEND);
-       } else {
+               break;
+       case WLSC_RGB_VISUAL:
                glDisable(GL_BLEND);
+               break;
+       default:
+               fprintf(stderr, "bogus visual\n");
+               break;
        }
 
        if (es->transform == NULL) {
@@ -686,6 +711,7 @@ wlsc_surface_raise(struct wlsc_surface *surface)
 
        wl_list_remove(&surface->link);
        wl_list_insert(&compositor->surface_list, &surface->link);
+       wlsc_compositor_repick(compositor);
        wlsc_surface_damage(surface);
 }
 
@@ -704,10 +730,8 @@ wlsc_buffer_post_release(struct wl_buffer *buffer)
        if (buffer == NULL || --buffer->busy_count > 0)
                return;
 
-       assert(buffer->client != NULL);
-       wl_client_post_event(buffer->client,
-                            &buffer->resource.object,
-                            WL_BUFFER_RELEASE);
+       assert(buffer->resource.client != NULL);
+       wl_resource_post_event(&buffer->resource, WL_BUFFER_RELEASE);
 }
 
 WL_EXPORT void
@@ -754,16 +778,16 @@ fade_output(struct wlsc_output *output,
        surface.compositor = compositor;
        surface.x = output->x;
        surface.y = output->y;
+       surface.pitch = output->current->width;
        surface.width = output->current->width;
        surface.height = output->current->height;
        surface.texture = GL_NONE;
        surface.transform = NULL;
 
        if (tint <= 1.0)
-               surface.visual =
-                       &compositor->compositor.premultiplied_argb_visual;
+               surface.visual = WLSC_PREMUL_ARGB_VISUAL;
        else
-               surface.visual = &compositor->compositor.rgb_visual;
+               surface.visual = WLSC_RGB_VISUAL;
 
        glUseProgram(compositor->solid_shader.program);
        glUniformMatrix4fv(compositor->solid_shader.proj_uniform,
@@ -795,8 +819,10 @@ wlsc_output_set_cursor(struct wlsc_output *output,
 
        prior_was_hardware = wl_list_empty(&device->sprite->link);
        if (force_sw || output->set_hardware_cursor(output, device) < 0) {
-               if (prior_was_hardware)
+               if (prior_was_hardware) {
                        wlsc_surface_damage(device->sprite);
+                       output->set_hardware_cursor(output, NULL);
+               }
                use_hardware_cursor = 0;
        } else if (!prior_was_hardware) {
                wlsc_surface_damage_below(device->sprite);
@@ -804,7 +830,7 @@ wlsc_output_set_cursor(struct wlsc_output *output,
 
        /* Remove always to be on top. */
        wl_list_remove(&device->sprite->link);
-       if (!use_hardware_cursor)
+       if (!use_hardware_cursor && ec->focus)
                wl_list_insert(&ec->surface_list, &device->sprite->link);
        else
                wl_list_init(&device->sprite->link);
@@ -830,7 +856,7 @@ wlsc_output_repaint(struct wlsc_output *output)
        glUniform1i(ec->texture_shader.tex_uniform, 0);
 
        wlsc_output_set_cursor(output, ec->input_device,
-                              !(ec->focus && ec->fade.spring.current < 0.001));
+                              ec->fade.spring.current >= 0.001);
 
        pixman_region32_init(&new_damage);
        pixman_region32_copy(&new_damage, &ec->damage);
@@ -855,7 +881,7 @@ wlsc_output_repaint(struct wlsc_output *output)
 
        es = container_of(ec->surface_list.next, struct wlsc_surface, link);
 
-       if (es->visual == &ec->compositor.rgb_visual &&
+       if (es->visual == WLSC_RGB_VISUAL &&
            output->prepare_scanout_surface(output, es) == 0) {
                /* We're drawing nothing now,
                 * draw the damaged regions later. */
@@ -896,25 +922,27 @@ wlsc_output_repaint(struct wlsc_output *output)
                fade_output(output, ec->fade.spring.current, &total_damage);
 }
 
+struct wlsc_frame_callback {
+       struct wl_resource resource;
+       struct wl_list link;
+};
+
 static void
 repaint(void *data, int msecs)
 {
        struct wlsc_output *output = data;
        struct wlsc_compositor *compositor = output->compositor;
-       struct wlsc_surface *es;
        struct wlsc_animation *animation, *next;
+       struct wlsc_frame_callback *cb, *cnext;
 
        wlsc_output_repaint(output);
        output->repaint_needed = 0;
        output->repaint_scheduled = 1;
        output->present(output);
 
-       /* FIXME: Keep the surfaces in an per-output list. */
-       wl_list_for_each(es, &compositor->surface_list, link) {
-               if (es->output == output) {
-                       wl_display_post_frame(compositor->wl_display,
-                                             &es->surface, msecs);
-               }
+       wl_list_for_each_safe(cb, cnext, &output->frame_callback_list, link) {
+               wl_resource_post_event(&cb->resource, WL_CALLBACK_DONE, msecs);
+               wl_resource_destroy(&cb->resource, 0);
        }
 
        wl_list_for_each_safe(animation, next,
@@ -980,11 +1008,9 @@ wlsc_compositor_fade(struct wlsc_compositor *compositor, float tint)
 }
 
 static void
-surface_destroy(struct wl_client *client,
-               struct wl_surface *surface)
+surface_destroy(struct wl_client *client, struct wl_resource *resource)
 {
-       wl_resource_destroy(&surface->resource, client,
-                           wlsc_compositor_get_time());
+       wl_resource_destroy(resource, wlsc_compositor_get_time());
 }
 
 WL_EXPORT void
@@ -992,33 +1018,34 @@ wlsc_surface_assign_output(struct wlsc_surface *es)
 {
        struct wlsc_compositor *ec = es->compositor;
        struct wlsc_output *output;
+       pixman_region32_t region;
+       uint32_t max, area;
+       pixman_box32_t *e;
 
-       struct wlsc_output *tmp = es->output;
-       es->output = NULL;
-
+       max = 0;
        wl_list_for_each(output, &ec->output_list, link) {
-               if (pixman_region32_contains_point(&output->region,
-                                                  es->x, es->y, NULL)) {
-                       if (output != tmp)
-                               printf("assiging surface %p to output %p\n",
-                                      es, output);
+               pixman_region32_init_rect(&region,
+                                         es->x, es->y, es->width, es->height);
+               pixman_region32_intersect(&region, &region, &output->region);
+
+               e = pixman_region32_extents(&region);
+               area = (e->x2 - e->x1) * (e->y2 - e->y1);
+
+               if (area >= max) {
                        es->output = output;
+                       max = area;
                }
        }
-       
-       if (es->output == NULL) {
-               printf("no output found\n");
-               es->output = container_of(ec->output_list.next,
-                                         struct wlsc_output, link);
-       }
 }
 
 static void
 surface_attach(struct wl_client *client,
-              struct wl_surface *surface, struct wl_buffer *buffer,
-              int32_t x, int32_t y)
+              struct wl_resource *resource,
+              struct wl_resource *buffer_resource, int32_t x, int32_t y)
 {
-       struct wlsc_surface *es = (struct wlsc_surface *) surface;
+       struct wlsc_surface *es = resource->data;
+       struct wl_buffer *buffer = buffer_resource->data;
+       int repick = 0;
 
        buffer->busy_count++;
        wlsc_buffer_post_release(es->buffer);
@@ -1028,35 +1055,80 @@ surface_attach(struct wl_client *client,
        wl_list_insert(es->buffer->resource.destroy_listener_list.prev,
                       &es->buffer_destroy_listener.link);
 
-       if (es->visual == NULL)
-               fprintf(stderr, "initial attach for surface %p\n", surface);
-
-       if (es->visual == NULL)
+       if (es->visual == WLSC_NONE_VISUAL) {
                wl_list_insert(&es->compositor->surface_list, &es->link);
+               repick = 1;
+       }
 
-       es->visual = buffer->visual;
-       wlsc_surface_configure(es, es->x + x, es->y + y,
-                              buffer->width, buffer->height);
+       if (x != 0 || y != 0 ||
+           es->width != buffer->width || es->height != buffer->height) {
+               wlsc_surface_configure(es, es->x + x, es->y + y,
+                                      buffer->width, buffer->height);
+               repick = 1;
+       }
 
-       wlsc_buffer_attach(buffer, surface);
+       wlsc_buffer_attach(buffer, &es->surface);
 
        es->compositor->shell->attach(es->compositor->shell, es);
+
+       if (repick)
+               wlsc_compositor_repick(es->compositor);
 }
 
 static void
 surface_damage(struct wl_client *client,
-              struct wl_surface *surface,
+              struct wl_resource *resource,
               int32_t x, int32_t y, int32_t width, int32_t height)
 {
-       struct wlsc_surface *es = (struct wlsc_surface *) surface;
+       struct wlsc_surface *es = resource->data;
 
        wlsc_surface_damage_rectangle(es, x, y, width, height);
 }
 
+static void
+destroy_frame_callback(struct wl_resource *resource)
+{
+       struct wlsc_frame_callback *cb = resource->data;
+
+       wl_list_remove(&cb->link);
+       free(resource);
+}
+
+static void
+surface_frame(struct wl_client *client,
+             struct wl_resource *resource, uint32_t callback)
+{
+       struct wlsc_frame_callback *cb;
+       struct wlsc_surface *es = resource->data;
+
+       cb = malloc(sizeof *cb);
+       if (cb == NULL) {
+               wl_resource_post_no_memory(resource);
+               return;
+       }
+               
+       cb->resource.object.interface = &wl_callback_interface;
+       cb->resource.object.id = callback;
+       cb->resource.destroy = destroy_frame_callback;
+       cb->resource.client = client;
+       cb->resource.data = cb;
+
+       wl_client_add_resource(client, &cb->resource);
+
+       if (es->output) {
+               wl_list_insert(es->output->frame_callback_list.prev,
+                              &cb->link);
+       } else {
+               wl_list_init(&cb->link);
+               wl_resource_destroy(&cb->resource, 0);
+       }
+}
+
 const static struct wl_surface_interface surface_interface = {
        surface_destroy,
        surface_attach,
-       surface_damage
+       surface_damage,
+       surface_frame
 };
 
 static void
@@ -1088,6 +1160,9 @@ static void
 wlsc_input_device_attach_sprite(struct wlsc_input_device *device,
                                struct wlsc_sprite *sprite, int x, int y)
 {
+       if (!sprite)
+               return;
+
        wlsc_sprite_attach(sprite, &device->sprite->surface);
        wlsc_input_device_attach(device, x, y, sprite->width, sprite->height);
 }
@@ -1107,14 +1182,14 @@ wlsc_input_device_set_pointer_image(struct wlsc_input_device *device,
 
 static void
 compositor_create_surface(struct wl_client *client,
-                         struct wl_compositor *compositor, uint32_t id)
+                         struct wl_resource *resource, uint32_t id)
 {
-       struct wlsc_compositor *ec = (struct wlsc_compositor *) compositor;
+       struct wlsc_compositor *ec = resource->data;
        struct wlsc_surface *surface;
 
        surface = wlsc_surface_create(ec, 0, 0, 0, 0);
        if (surface == NULL) {
-               wl_client_post_no_memory(client);
+               wl_resource_post_no_memory(resource);
                return;
        }
 
@@ -1124,7 +1199,7 @@ compositor_create_surface(struct wl_client *client,
        surface->surface.resource.object.interface = &wl_surface_interface;
        surface->surface.resource.object.implementation =
                (void (**)(void)) &surface_interface;
-       surface->surface.client = client;
+       surface->surface.resource.data = surface;
 
        wl_client_add_resource(client, &surface->surface.resource);
 }
@@ -1149,7 +1224,7 @@ pick_surface(struct wl_input_device *device, int32_t *sx, int32_t *sy)
        struct wlsc_surface *es;
 
        wl_list_for_each(es, &ec->surface_list, link) {
-               if (es->surface.client == NULL)
+               if (es->surface.resource.client == NULL)
                        continue;
                wlsc_surface_transform(es, device->x, device->y, sx, sy);
                if (0 <= *sx && *sx < es->width &&
@@ -1162,41 +1237,45 @@ pick_surface(struct wl_input_device *device, int32_t *sx, int32_t *sy)
 
 
 static void
-motion_grab_motion(struct wl_grab *grab,
-                  uint32_t time, int32_t x, int32_t y)
+implicit_grab_motion(struct wl_grab *grab,
+                    uint32_t time, int32_t x, int32_t y)
 {
        struct wlsc_input_device *device =
                (struct wlsc_input_device *) grab->input_device;
        struct wlsc_surface *es =
                (struct wlsc_surface *) device->input_device.pointer_focus;
        int32_t sx, sy;
+       struct wl_resource *resource;
 
-       wlsc_surface_transform(es, x, y, &sx, &sy);
-       wl_client_post_event(es->surface.client,
-                            &device->input_device.object,
-                            WL_INPUT_DEVICE_MOTION,
-                            time, x, y, sx, sy);
+       resource = grab->input_device->pointer_focus_resource;
+       if (resource) {
+               wlsc_surface_transform(es, x, y, &sx, &sy);
+               wl_resource_post_event(resource, WL_INPUT_DEVICE_MOTION,
+                                      time, x, y, sx, sy);
+       }
 }
 
 static void
-motion_grab_button(struct wl_grab *grab,
-                  uint32_t time, int32_t button, int32_t state)
+implicit_grab_button(struct wl_grab *grab,
+                    uint32_t time, int32_t button, int32_t state)
 {
-       wl_client_post_event(grab->input_device->pointer_focus->client,
-                            &grab->input_device->object,
-                            WL_INPUT_DEVICE_BUTTON,
-                            time, button, state);
+       struct wl_resource *resource;
+
+       resource = grab->input_device->pointer_focus_resource;
+       if (resource)
+               wl_resource_post_event(resource, WL_INPUT_DEVICE_BUTTON,
+                                      time, button, state);
 }
 
 static void
-motion_grab_end(struct wl_grab *grab, uint32_t time)
+implicit_grab_end(struct wl_grab *grab, uint32_t time)
 {
 }
 
-static const struct wl_grab_interface motion_grab_interface = {
-       motion_grab_motion,
-       motion_grab_button,
-       motion_grab_end
+static const struct wl_grab_interface implicit_grab_interface = {
+       implicit_grab_motion,
+       implicit_grab_button,
+       implicit_grab_end
 };
 
 WL_EXPORT void
@@ -1297,11 +1376,10 @@ notify_motion(struct wl_input_device *device, uint32_t time, int x, int y)
                wl_input_device_set_pointer_focus(device,
                                                  &es->surface,
                                                  time, x, y, sx, sy);
-               if (es)
-                       wl_client_post_event(es->surface.client,
-                                            &device->object,
-                                            WL_INPUT_DEVICE_MOTION,
-                                            time, x, y, sx, sy);
+               if (device->pointer_focus_resource)
+                       wl_resource_post_event(device->pointer_focus_resource,
+                                              WL_INPUT_DEVICE_MOTION,
+                                              time, x, y, sx, sy);
        }
 
        wlsc_surface_damage_below(wd->sprite);
@@ -1348,6 +1426,7 @@ notify_button(struct wl_input_device *device,
        struct wlsc_binding *b;
        struct wlsc_surface *surface =
                (struct wlsc_surface *) device->pointer_focus;
+       int32_t sx, sy;
 
        if (state)
                wlsc_compositor_idle_inhibit(compositor);
@@ -1355,9 +1434,10 @@ notify_button(struct wl_input_device *device,
                wlsc_compositor_idle_release(compositor);
 
        if (state && surface && device->grab == NULL) {
-               wlsc_surface_activate(surface, wd, time);
+               compositor->shell->activate(compositor->shell,
+                                           surface, wd, time);
                wl_input_device_start_grab(device,
-                                          &device->motion_grab,
+                                          &device->implicit_grab,
                                           button, time);
        }
 
@@ -1374,8 +1454,13 @@ notify_button(struct wl_input_device *device,
                device->grab->interface->button(device->grab, time,
                                                button, state);
 
-       if (!state && device->grab && device->grab_button == button)
+       if (!state && device->grab && device->grab_button == button) {
                wl_input_device_end_grab(device, time);
+               surface = pick_surface(device, &sx, &sy);
+               wl_input_device_set_pointer_focus(device, &surface->surface,
+                                                 time, device->x, device->y,
+                                                 sx, sy);
+       }
 }
 
 static void
@@ -1485,10 +1570,9 @@ notify_key(struct wl_input_device *device,
                *k = key;
        }
 
-       if (device->keyboard_focus != NULL)
-               wl_client_post_event(device->keyboard_focus->client,
-                                    &device->object,
-                                    WL_INPUT_DEVICE_KEY, time, key, state);
+       if (device->keyboard_focus_resource)
+               wl_resource_post_event(device->keyboard_focus_resource,
+                                      WL_INPUT_DEVICE_KEY, time, key, state);
 }
 
 WL_EXPORT void
@@ -1550,7 +1634,7 @@ notify_keyboard_focus(struct wl_input_device *device,
                        update_modifier_state(wd, *k, 1);
                }
 
-               if (es->surface.client)
+               if (es && es->surface.resource.client)
                        wl_input_device_set_keyboard_focus(&wd->input_device,
                                                           &es->surface, time);
        } else {
@@ -1567,44 +1651,61 @@ notify_keyboard_focus(struct wl_input_device *device,
 
 static void
 input_device_attach(struct wl_client *client,
-                   struct wl_input_device *device_base,
+                   struct wl_resource *resource,
                    uint32_t time,
-                   struct wl_buffer *buffer, int32_t x, int32_t y)
+                   struct wl_resource *buffer_resource, int32_t x, int32_t y)
 {
-       struct wlsc_input_device *device =
-               (struct wlsc_input_device *) device_base;
+       struct wlsc_input_device *device = resource->data;
+       struct wl_buffer *buffer;
 
        if (time < device->input_device.pointer_focus_time)
                return;
        if (device->input_device.pointer_focus == NULL)
                return;
-       if (device->input_device.pointer_focus->client != client)
+       if (device->input_device.pointer_focus->resource.client != client)
                return;
 
-       if (buffer == NULL) {
+       if (buffer_resource) {
+               buffer = buffer_resource->data;
+               wlsc_input_device_attach_buffer(device, buffer, x, y);
+       } else {
                wlsc_input_device_set_pointer_image(device,
                                                    WLSC_POINTER_LEFT_PTR);
                return;
        }
-
-       wlsc_input_device_attach_buffer(device, buffer, x, y);
 }
 
 const static struct wl_input_device_interface input_device_interface = {
        input_device_attach,
 };
 
+static void unbind_input_device(struct wl_resource *resource)
+{
+       wl_list_remove(&resource->link);
+       free(resource);
+}
+
+static void
+bind_input_device(struct wl_client *client,
+                 void *data, uint32_t version, uint32_t id)
+{
+       struct wl_input_device *device = data;
+       struct wl_resource *resource;
+
+       resource = wl_client_add_object(client, &wl_input_device_interface,
+                                       &input_device_interface, id, data);
+       wl_list_insert(&device->resource_list, &resource->link);
+       resource->destroy = unbind_input_device;
+}
+
 WL_EXPORT void
 wlsc_input_device_init(struct wlsc_input_device *device,
                       struct wlsc_compositor *ec)
 {
        wl_input_device_init(&device->input_device, &ec->compositor);
 
-       device->input_device.object.interface = &wl_input_device_interface;
-       device->input_device.object.implementation =
-               (void (**)(void)) &input_device_interface;
-       wl_display_add_object(ec->wl_display, &device->input_device.object);
-       wl_display_add_global(ec->wl_display, &device->input_device.object, NULL);
+       wl_display_add_global(ec->wl_display, &wl_input_device_interface,
+                             device, bind_input_device);
 
        device->sprite = wlsc_surface_create(ec,
                                             device->input_device.x,
@@ -1615,7 +1716,7 @@ wlsc_input_device_init(struct wlsc_input_device *device,
        device->hotspot_y = 16;
        device->modifier_state = 0;
 
-       device->input_device.motion_grab.interface = &motion_grab_interface;
+       device->input_device.implicit_grab.interface = &implicit_grab_interface;
 
        wl_list_insert(ec->input_device_list.prev, &device->link);
 
@@ -1623,27 +1724,32 @@ wlsc_input_device_init(struct wlsc_input_device *device,
 }
 
 static void
-wlsc_output_post_geometry(struct wl_client *client,
-                         struct wl_object *global, uint32_t version)
+bind_output(struct wl_client *client,
+           void *data, uint32_t version, uint32_t id)
 {
-       struct wlsc_output *output =
-               container_of(global, struct wlsc_output, object);
+       struct wlsc_output *output = data;
        struct wlsc_mode *mode;
+       struct wl_resource *resource;
 
-       wl_client_post_event(client, global,
-                            WL_OUTPUT_GEOMETRY,
-                            output->x,
-                            output->y,
-                            output->mm_width,
-                            output->mm_height,
-                            output->subpixel,
-                            output->make, output->model);
+       resource = wl_client_add_object(client,
+                                       &wl_output_interface, NULL, id, data);
+
+       wl_resource_post_event(resource,
+                              WL_OUTPUT_GEOMETRY,
+                              output->x,
+                              output->y,
+                              output->mm_width,
+                              output->mm_height,
+                              output->subpixel,
+                              output->make, output->model);
 
        wl_list_for_each (mode, &output->mode_list, link) {
-               wl_client_post_event(client, global,
-                                    WL_OUTPUT_MODE,
-                                    mode->flags,
-                                    mode->width, mode->height, mode->refresh);
+               wl_resource_post_event(resource,
+                                      WL_OUTPUT_MODE,
+                                      mode->flags,
+                                      mode->width,
+                                      mode->height,
+                                      mode->refresh);
        }
 }
 
@@ -1763,7 +1869,6 @@ wlsc_output_destroy(struct wlsc_output *output)
 {
        pixman_region32_fini(&output->region);
        pixman_region32_fini(&output->previous_damage);
-       destroy_surface(&output->background->surface.resource, NULL);
 }
 
 WL_EXPORT void
@@ -1775,11 +1880,6 @@ wlsc_output_move(struct wlsc_output *output, int x, int y)
        output->x = x;
        output->y = y;
 
-       if (output->background) {
-               output->background->x = x;
-               output->background->y = y;
-       }
-
        pixman_region32_init(&output->previous_damage);
        pixman_region32_init_rect(&output->region, x, y, 
                                  output->current->width,
@@ -1808,24 +1908,17 @@ wlsc_output_init(struct wlsc_output *output, struct wlsc_compositor *c,
        output->mm_width = width;
        output->mm_height = height;
 
-       output->background =
-               background_create(output, option_background);
-       if (output->background != NULL)
-               wl_list_insert(c->surface_list.prev,
-                              &output->background->link);
-
        output->flags = flags;
        wlsc_output_move(output, x, y);
 
        output->scanout_buffer_destroy_listener.func =
                output_handle_scanout_buffer_destroy;
        wl_list_init(&output->scanout_buffer_destroy_listener.link);
+       wl_list_init(&output->frame_callback_list);
 
-       output->object.interface = &wl_output_interface;
-       wl_display_add_object(c->wl_display, &output->object);
-       wl_display_add_global(c->wl_display, &output->object,
-                             wlsc_output_post_geometry);
+       output->resource.object.interface = &wl_output_interface;
+       wl_display_add_global(c->wl_display,
+                             &wl_output_interface, output, bind_output);
 }
 
 static void
@@ -1957,12 +2050,22 @@ wlsc_compositor_init(struct wlsc_compositor *ec, struct wl_display *display)
        return 0;
 }
 
+WL_EXPORT void
+wlsc_compositor_shutdown(struct wlsc_compositor *ec)
+{
+       struct wlsc_output *output;
+
+       /* Destroy all outputs associated with this compositor */
+       wl_list_for_each(output, &ec->output_list, link)
+               output->destroy(output);
+}
+
 static int on_term_signal(int signal_number, void *data)
 {
-       struct wlsc_compositor *ec = data;
+       struct wl_display *display = data;
 
        fprintf(stderr, "caught signal %d\n", signal_number);
-       wl_display_terminate(ec->wl_display);
+       wl_display_terminate(display);
 
        return 1;
 }
@@ -2014,7 +2117,6 @@ int main(int argc, char *argv[])
        static const struct option longopts[ ] = {
                { "backend", 1, NULL, 'B' },
                { "backend-options", 1, NULL, 'o' },
-               { "background", 1, NULL, 'b' },
                { "socket", 1, NULL, 'S' },
                { "idle-time", 1, NULL, 'i' },
                { "shell", 1, NULL, 's' },
@@ -2024,9 +2126,6 @@ int main(int argc, char *argv[])
 
        while (o = getopt_long(argc, argv, opts, longopts, &o), o > 0) {
                switch (o) {
-               case 'b':
-                       option_background = optarg;
-                       break;
                case 'B':
                        backend = optarg;
                        break;
@@ -2056,7 +2155,13 @@ int main(int argc, char *argv[])
 
        display = wl_display_create();
 
-       ec = NULL;
+       loop = wl_display_get_event_loop(display);
+       wl_event_loop_add_signal(loop, SIGTERM, on_term_signal, display);
+       wl_event_loop_add_signal(loop, SIGINT, on_term_signal, display);
+       wl_event_loop_add_signal(loop, SIGQUIT, on_term_signal, display);
+
+       wl_list_init(&child_process_list);
+       wl_event_loop_add_signal(loop, SIGCHLD, sigchld_handler, NULL);
 
        if (!backend) {
                if (getenv("WAYLAND_DISPLAY"))
@@ -2097,14 +2202,6 @@ int main(int argc, char *argv[])
                exit(EXIT_FAILURE);
        }
 
-       loop = wl_display_get_event_loop(ec->wl_display);
-       wl_event_loop_add_signal(loop, SIGTERM, on_term_signal, ec);
-       wl_event_loop_add_signal(loop, SIGINT, on_term_signal, ec);
-       wl_event_loop_add_signal(loop, SIGQUIT, on_term_signal, ec);
-
-       wl_list_init(&child_process_list);
-       wl_event_loop_add_signal(loop, SIGCHLD, sigchld_handler, NULL);
-
        wl_display_run(display);
 
        if (xserver)