wayland: simpler layout for controller::proxy_to_id maps
[staging/windowmanager.git] / src / wayland.cpp
index c19b0dc..af28edf 100644 (file)
@@ -1,3 +1,5 @@
+#include <utility>
+
 #include "wayland.hpp"
 
 //                                                                  _
@@ -15,16 +17,20 @@ namespace wl {
 //  \__,_|_|___/ .__/|_|\__,_|\__, |
 //             |_|            |___/
 display::display()
-   : d(std::unique_ptr<struct wl_display,
-                       std::function<void(struct wl_display *)>>(
-        wl_display_connect(NULL),
+   : d(std::unique_ptr<struct wl_display, void (*)(struct wl_display *)>(
+        wl_display_connect(nullptr),
+#ifdef DEBUG_OUTPUT
         [](struct wl_display *d) {
            logdebug("wl::display ~display @ %p", d);
            wl_display_disconnect(d);
         })),
-     r(!d ? nullptr : std::make_unique<struct registry>(d.get())) {}
+#else
+        &wl_display_disconnect)),
+#endif
+     r(d.get()) {
+}
 
-display::~display() {}
+display::~display() = default;
 
 bool display::ok() const { return d && wl_display_get_error(d.get()) == 0; }
 
@@ -32,6 +38,10 @@ void display::roundtrip() { wl_display_roundtrip(this->d.get()); }
 
 int display::dispatch() { return wl_display_dispatch(this->d.get()); }
 
+void display::flush() { wl_display_flush(this->d.get()); }
+
+int display::get_fd() const { return wl_display_get_fd(this->d.get()); }
+
 //                 _     _
 //  _ __ ___  __ _(_)___| |_ _ __ _   _
 // | '__/ _ \/ _` | / __| __| '__| | | |
@@ -39,22 +49,25 @@ int display::dispatch() { return wl_display_dispatch(this->d.get()); }
 // |_|  \___|\__, |_|___/\__|_|   \__, |
 //           |___/                |___/
 namespace {
-void registry_global(void *data, struct wl_registry *r, uint32_t name,
+void registry_global(void *data, struct wl_registry * /*r*/, uint32_t name,
                      char const *iface, uint32_t v) {
    static_cast<struct registry *>(data)->global(name, iface, v);
 }
 
-void registry_global_remove(void *data, struct wl_registry *r, uint32_t name) {
+void registry_global_remove(void *data, struct wl_registry * /*r*/,
+                            uint32_t name) {
    static_cast<struct registry *>(data)->global_remove(name);
 }
 
 constexpr struct wl_registry_listener registry_listener = {
    registry_global, registry_global_remove};
-}
+}  // namespace
 
 registry::registry(struct wl_display *d)
-   : wayland_proxy(wl_display_get_registry(d)) {
-   wl_registry_add_listener(this->proxy, &registry_listener, this);
+   : wayland_proxy(d == nullptr ? nullptr : wl_display_get_registry(d)) {
+   if (this->proxy != nullptr) {
+      wl_registry_add_listener(this->proxy, &registry_listener, this);
+   }
 }
 
 registry::~registry() {
@@ -62,19 +75,19 @@ registry::~registry() {
 }
 
 void registry::add_global_handler(char const *iface, binder bind) {
-   this->bindings[iface] = bind;
+   this->bindings[iface] = std::move(bind);
 }
 
 void registry::global(uint32_t name, char const *iface, uint32_t v) {
    auto b = this->bindings.find(iface);
-   if (b != this->bindings.end())
+   if (b != this->bindings.end()) {
       b->second(this->proxy, name, v);
-   else
-      logdebug("wl::registry @ %p global n %u i %s v %u", this->proxy, name,
-               iface, v);
+   }
+   logdebug("wl::registry @ %p global n %u i %s v %u", this->proxy, name, iface,
+            v);
 }
 
-void registry::global_remove(uint32_t name) {}
+void registry::global_remove(uint32_t /*name*/) {}
 
 //              _               _
 //   ___  _   _| |_ _ __  _   _| |_
@@ -83,7 +96,7 @@ void registry::global_remove(uint32_t name) {}
 //  \___/ \__,_|\__| .__/ \__,_|\__|
 //                 |_|
 namespace {
-void output_geometry(void *data, struct wl_output *wl_output, int32_t x,
+void output_geometry(void *data, struct wl_output * /*wl_output*/, int32_t x,
                      int32_t y, int32_t physical_width, int32_t physical_height,
                      int32_t subpixel, const char *make, const char *model,
                      int32_t transform) {
@@ -91,22 +104,23 @@ void output_geometry(void *data, struct wl_output *wl_output, int32_t x,
       x, y, physical_width, physical_height, subpixel, make, model, transform);
 }
 
-void output_mode(void *data, struct wl_output *wl_output, uint32_t flags,
+void output_mode(void *data, struct wl_output * /*wl_output*/, uint32_t flags,
                  int32_t width, int32_t height, int32_t refresh) {
    static_cast<struct output *>(data)->mode(flags, width, height, refresh);
 }
 
-void output_done(void *data, struct wl_output *wl_output) {
+void output_done(void *data, struct wl_output * /*wl_output*/) {
    static_cast<struct output *>(data)->done();
 }
 
-void output_scale(void *data, struct wl_output *wl_output, int32_t factor) {
+void output_scale(void *data, struct wl_output * /*wl_output*/,
+                  int32_t factor) {
    static_cast<struct output *>(data)->scale(factor);
 }
 
 constexpr struct wl_output_listener output_listener = {
    output_geometry, output_mode, output_done, output_scale};
-}
+}  // namespace
 
 output::output(struct wl_registry *r, uint32_t name, uint32_t v)
    : wayland_proxy(wl_registry_bind(r, name, &wl_output_interface, v)) {
@@ -124,6 +138,11 @@ void output::geometry(int32_t x, int32_t y, int32_t pw, int32_t ph,
 void output::mode(uint32_t flags, int32_t w, int32_t h, int32_t r) {
    logdebug("wl::output %s @ %p f %x w %i h %i r %i", __func__, this->proxy,
             flags, w, h, r);
+   if ((flags & WL_OUTPUT_MODE_CURRENT) != 0u) {
+      this->width = w;
+      this->height = h;
+      this->refresh = r;
+   }
 }
 
 void output::done() {
@@ -133,7 +152,7 @@ void output::done() {
 void output::scale(int32_t factor) {
    logdebug("wl::output %s @ %p f %i", __func__, this->proxy, factor);
 }
-}
+}  // namespace wl
 
 //  _ __   __ _ _ __ ___   ___  ___ _ __   __ _  ___ ___
 // | '_ \ / _` | '_ ` _ \ / _ \/ __| '_ \ / _` |/ __/ _ \
@@ -155,23 +174,23 @@ namespace genivi {
 //  \___\___/|_| |_|\__|_|  \___/|_|_|\___|_|
 //
 namespace {
-void controller_screen(void *data, struct ivi_controller *ivi_controller,
+void controller_screen(void *data, struct ivi_controller * /*ivi_controller*/,
                        uint32_t id_screen,
                        struct ivi_controller_screen *screen) {
    static_cast<struct controller *>(data)->controller_screen(id_screen, screen);
 }
 
-void controller_layer(void *data, struct ivi_controller *ivi_controller,
+void controller_layer(void *data, struct ivi_controller * /*ivi_controller*/,
                       uint32_t id_layer) {
    static_cast<struct controller *>(data)->controller_layer(id_layer);
 }
 
-void controller_surface(void *data, struct ivi_controller *ivi_controller,
+void controller_surface(void *data, struct ivi_controller * /*ivi_controller*/,
                         uint32_t id_surface) {
    static_cast<struct controller *>(data)->controller_surface(id_surface);
 }
 
-void controller_error(void *data, struct ivi_controller *ivi_controller,
+void controller_error(void *data, struct ivi_controller * /*ivi_controller*/,
                       int32_t object_id, int32_t object_type,
                       int32_t error_code, const char *error_text) {
    static_cast<struct controller *>(data)->controller_error(
@@ -180,15 +199,31 @@ void controller_error(void *data, struct ivi_controller *ivi_controller,
 
 constexpr struct ivi_controller_listener listener = {
    controller_screen, controller_layer, controller_surface, controller_error};
-}
+}  // namespace
 
 controller::controller(struct wl_registry *r, uint32_t name, uint32_t version)
    : wayland_proxy(
-        wl_registry_bind(r, name, &ivi_controller_interface, version)) {
+        wl_registry_bind(r, name, &ivi_controller_interface, version)),
+     surface_proxy_to_id{},
+     layer_proxy_to_id{},
+     screen_proxy_to_id{},
+     surfaces{},
+     layers{},
+     screens{},
+     pending{},
+     output_size{} {
    ivi_controller_add_listener(this->proxy, &listener, this);
 }
 
-controller::~controller() {}
+controller::~controller() = default;
+
+void controller::layer_create(uint32_t id, int32_t w, int32_t h) {
+   this->layers[id] = std::make_unique<layer>(id, w, h, this);
+}
+
+void controller::surface_create(uint32_t id) {
+   this->surfaces[id] = std::make_unique<surface>(id, this);
+}
 
 void controller::controller_screen(uint32_t id,
                                    struct ivi_controller_screen *screen) {
@@ -199,12 +234,22 @@ void controller::controller_screen(uint32_t id,
 
 void controller::controller_layer(uint32_t id) {
    logdebug("genivi::controller @ %p layer %u (%x)", this->proxy, id, id);
-   this->layers[id] = std::make_unique<struct layer>(id, this);
+   auto &l = this->layers[id] = std::make_unique<struct layer>(id, this);
+   l->clear_surfaces();
 }
 
 void controller::controller_surface(uint32_t id) {
    logdebug("genivi::controller @ %p surface %u (%x)", this->proxy, id, id);
    this->surfaces[id] = std::make_unique<struct surface>(id, this);
+
+   add_task("fullscreen surface", [id](struct controller *c) {
+      auto &s = c->surfaces[id];
+      s->set_destination_rectangle(0, 0, c->output_size.w, c->output_size.h);
+      s->set_visibility(1);
+      uint32_t lid = id == 0x16180 ? 1000 : 100;
+      c->layers[lid]->add_surface(s.get());
+      logdebug("Surface %u now fullscreen on layer %u", id, lid);
+   });
 }
 
 void controller::controller_error(int32_t object_id, int32_t object_type,
@@ -221,58 +266,58 @@ void controller::controller_error(int32_t object_id, int32_t object_type,
 //          |___/
 namespace {
 void layer_visibility(void *data,
-                      struct ivi_controller_layer *ivi_controller_layer,
+                      struct ivi_controller_layer * /*ivi_controller_layer*/,
                       int32_t visibility) {
-   static_cast<struct layer *>(data)->parent->layer_visibility(
-      static_cast<struct layer *>(data)->id, visibility);
+   auto l = static_cast<struct layer *>(data);
+   l->parent->layer_visibility(l->id, visibility);
 }
 
 void layer_opacity(void *data,
-                   struct ivi_controller_layer *ivi_controller_layer,
+                   struct ivi_controller_layer * /*ivi_controller_layer*/,
                    wl_fixed_t opacity) {
-   static_cast<struct layer *>(data)->parent->layer_opacity(
-      static_cast<struct layer *>(data)->id, wl_fixed_to_double(opacity));
+   auto l = static_cast<struct layer *>(data);
+   l->parent->layer_opacity(l->id, float(wl_fixed_to_double(opacity)));
 }
 
-void layer_source_rectangle(void *data,
-                            struct ivi_controller_layer *ivi_controller_layer,
-                            int32_t x, int32_t y, int32_t width,
-                            int32_t height) {
-   static_cast<struct layer *>(data)->parent->layer_source_rectangle(
-      static_cast<struct layer *>(data)->id, x, y, width, height);
+void layer_source_rectangle(
+   void *data, struct ivi_controller_layer * /*ivi_controller_layer*/,
+   int32_t x, int32_t y, int32_t width, int32_t height) {
+   auto l = static_cast<struct layer *>(data);
+   l->parent->layer_source_rectangle(l->id, x, y, width, height);
 }
 
 void layer_destination_rectangle(
-   void *data, struct ivi_controller_layer *ivi_controller_layer, int32_t x,
-   int32_t y, int32_t width, int32_t height) {
-   static_cast<struct layer *>(data)->parent->layer_destination_rectangle(
-      static_cast<struct layer *>(data)->id, x, y, width, height);
+   void *data, struct ivi_controller_layer * /*ivi_controller_layer*/,
+   int32_t x, int32_t y, int32_t width, int32_t height) {
+   auto l = static_cast<struct layer *>(data);
+   l->parent->layer_destination_rectangle(l->id, x, y, width, height);
 }
 
 void layer_configuration(void *data,
-                         struct ivi_controller_layer *ivi_controller_layer,
+                         struct ivi_controller_layer * /*ivi_controller_layer*/,
                          int32_t width, int32_t height) {
-   static_cast<struct layer *>(data)->parent->layer_configuration(
-      static_cast<struct layer *>(data)->id, width, height);
+   auto l = static_cast<struct layer *>(data);
+   l->parent->layer_configuration(l->id, width, height);
 }
 
 void layer_orientation(void *data,
-                       struct ivi_controller_layer *ivi_controller_layer,
+                       struct ivi_controller_layer * /*ivi_controller_layer*/,
                        int32_t orientation) {
-   static_cast<struct layer *>(data)->parent->layer_orientation(
-      static_cast<struct layer *>(data)->id, orientation);
+   auto l = static_cast<struct layer *>(data);
+   l->parent->layer_orientation(l->id, orientation);
 }
 
-void layer_screen(void *data, struct ivi_controller_layer *ivi_controller_layer,
+void layer_screen(void *data,
+                  struct ivi_controller_layer * /*ivi_controller_layer*/,
                   struct wl_output *screen) {
-   static_cast<struct layer *>(data)->parent->layer_screen(
-      static_cast<struct layer *>(data)->id, screen);
+   auto l = static_cast<struct layer *>(data);
+   l->parent->layer_screen(l->id, screen);
 }
 
 void layer_destroyed(void *data,
-                     struct ivi_controller_layer *ivi_controller_layer) {
-   static_cast<struct layer *>(data)->parent->layer_destroyed(
-      static_cast<struct layer *>(data)->id);
+                     struct ivi_controller_layer * /*ivi_controller_layer*/) {
+   auto l = static_cast<struct layer *>(data);
+   l->parent->layer_destroyed(l->id);
 }
 
 constexpr struct ivi_controller_layer_listener layer_listener = {
@@ -281,56 +326,125 @@ constexpr struct ivi_controller_layer_listener layer_listener = {
    layer_configuration,    layer_orientation,
    layer_screen,           layer_destroyed,
 };
-}
-
-layer::layer(uint32_t i, struct controller *c)
-   : wayland_proxy(ivi_controller_layer_create(c->proxy, i, 0, 0)),
-     controlled_entity(c, i) {
+}  // namespace
+
+layer::layer(uint32_t i, struct controller *c) : layer(i, 0, 0, c) {}
+
+layer::layer(uint32_t i, int32_t w, int32_t h, struct controller *c)
+   : wayland_proxy(ivi_controller_layer_create(c->proxy, i, w, h)),
+     controller_child(c, i),
+     dst_rect{},
+     src_rect{},
+     size{},
+     orientation{},
+     visibility{},
+     opacity{} {
+   this->parent->add_proxy_to_id_mapping(this->proxy, i);
    ivi_controller_layer_add_listener(this->proxy, &layer_listener, this);
 }
 
 layer::~layer() {
    logdebug("%s layer %i @ %p", __func__, this->id, this->proxy);
+   this->parent->remove_proxy_to_id_mapping(this->proxy);
    ivi_controller_layer_destroy(this->proxy, 1);
    this->proxy = nullptr;
 }
 
+void layer::set_visibility(uint32_t visibility) {
+   ivi_controller_layer_set_visibility(this->proxy, visibility);
+}
+
+void layer::set_opacity(wl_fixed_t opacity) {
+   ivi_controller_layer_set_opacity(this->proxy, opacity);
+}
+
+void layer::set_source_rectangle(int32_t x, int32_t y, int32_t width,
+                                 int32_t height) {
+   ivi_controller_layer_set_source_rectangle(this->proxy, x, y, width, height);
+}
+
+void layer::set_destination_rectangle(int32_t x, int32_t y, int32_t width,
+                                      int32_t height) {
+   ivi_controller_layer_set_destination_rectangle(this->proxy, x, y, width,
+                                                  height);
+}
+
+void layer::set_configuration(int32_t width, int32_t height) {
+   ivi_controller_layer_set_configuration(this->proxy, width, height);
+}
+
+void layer::set_orientation(int32_t orientation) {
+   ivi_controller_layer_set_orientation(this->proxy, orientation);
+}
+
+void layer::screenshot(const char *filename) {
+   ivi_controller_layer_screenshot(this->proxy, filename);
+}
+
+void layer::clear_surfaces() {
+   ivi_controller_layer_clear_surfaces(this->proxy);
+}
+
+void layer::add_surface(struct surface *surface) {
+   ivi_controller_layer_add_surface(this->proxy, surface->proxy);
+}
+
+void layer::remove_surface(struct surface *surface) {
+   ivi_controller_layer_remove_surface(this->proxy, surface->proxy);
+}
+
+void layer::set_render_order(std::vector<uint32_t> const &ro) {
+   struct wl_array wlro {
+      .size = ro.size() * sizeof(ro[0]), .alloc = ro.capacity() * sizeof(ro[0]),
+      .data = const_cast<void *>(static_cast<void const *>(ro.data()))
+   };
+   ivi_controller_layer_set_render_order(this->proxy, &wlro);
+}
+
 void controller::layer_visibility(uint32_t id, int32_t visibility) {
    logdebug("genivi::layer %s @ %p v %i", __func__, this->proxy, visibility);
+   this->layers[id]->visibility = visibility;
 }
 
 void controller::layer_opacity(uint32_t id, float opacity) {
    logdebug("genivi::layer %s @ %p o %f", __func__, this->proxy, opacity);
+   this->layers[id]->opacity = opacity;
 }
 
 void controller::layer_source_rectangle(uint32_t id, int32_t x, int32_t y,
                                         int32_t width, int32_t height) {
    logdebug("genivi::layer %s @ %p x %i y %i w %i h %i", __func__, this->proxy,
             x, y, width, height);
+   this->layers[id]->src_rect = rect{uint32_t(width), uint32_t(height), x, y};
 }
 
 void controller::layer_destination_rectangle(uint32_t id, int32_t x, int32_t y,
                                              int32_t width, int32_t height) {
    logdebug("genivi::layer %s @ %p x %i y %i w %i h %i", __func__, this->proxy,
             x, y, width, height);
+   this->layers[id]->dst_rect = rect{uint32_t(width), uint32_t(height), x, y};
 }
 
 void controller::layer_configuration(uint32_t id, int32_t width,
                                      int32_t height) {
    logdebug("genivi::layer %s @ %p w %i h %i", __func__, this->proxy, width,
             height);
+   this->layers[id]->size = size{uint32_t(width), uint32_t(height)};
 }
 
 void controller::layer_orientation(uint32_t id, int32_t orientation) {
    logdebug("genivi::layer %s @ %p o %i", __func__, this->proxy, orientation);
+   this->layers[id]->orientation = orientation;
 }
 
-void controller::layer_screen(uint32_t id, struct wl_output *screen) {
+void controller::layer_screen(uint32_t /*id*/, struct wl_output *screen) {
    logdebug("genivi::layer %s @ %p s %p", __func__, this->proxy, screen);
 }
 
 void controller::layer_destroyed(uint32_t id) {
    logdebug("genivi::layer %s @ %p", __func__, this->proxy);
+   add_task("remove layer",
+            [id](struct controller *c) { c->layers.erase(id); });
 }
 
 //                  __
@@ -341,83 +455,83 @@ void controller::layer_destroyed(uint32_t id) {
 //
 namespace {
 
-void surface_visibility(void *data,
-                        struct ivi_controller_surface *ivi_controller_surface,
-                        int32_t visibility) {
-   static_cast<struct surface *>(data)->parent->surface_visibility(
-      static_cast<struct surface *>(data)->id, visibility);
+void surface_visibility(
+   void *data, struct ivi_controller_surface * /*ivi_controller_surface*/,
+   int32_t visibility) {
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_visibility(s->id, visibility);
 }
 
 void surface_opacity(void *data,
-                     struct ivi_controller_surface *ivi_controller_surface,
+                     struct ivi_controller_surface * /*ivi_controller_surface*/,
                      wl_fixed_t opacity) {
-   static_cast<struct surface *>(data)->parent->surface_opacity(
-      static_cast<struct surface *>(data)->id, wl_fixed_to_double(opacity));
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_opacity(s->id, float(wl_fixed_to_double(opacity)));
 }
 
 void surface_source_rectangle(
-   void *data, struct ivi_controller_surface *ivi_controller_surface, int32_t x,
-   int32_t y, int32_t width, int32_t height) {
-   static_cast<struct surface *>(data)->parent->surface_source_rectangle(
-      static_cast<struct surface *>(data)->id, x, y, width, height);
+   void *data, struct ivi_controller_surface * /*ivi_controller_surface*/,
+   int32_t x, int32_t y, int32_t width, int32_t height) {
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_source_rectangle(s->id, x, y, width, height);
 }
 
 void surface_destination_rectangle(
-   void *data, struct ivi_controller_surface *ivi_controller_surface, int32_t x,
-   int32_t y, int32_t width, int32_t height) {
-   static_cast<struct surface *>(data)->parent->surface_destination_rectangle(
-      static_cast<struct surface *>(data)->id, x, y, width, height);
+   void *data, struct ivi_controller_surface * /*ivi_controller_surface*/,
+   int32_t x, int32_t y, int32_t width, int32_t height) {
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_destination_rectangle(s->id, x, y, width, height);
 }
 
 void surface_configuration(
-   void *data, struct ivi_controller_surface *ivi_controller_surface,
+   void *data, struct ivi_controller_surface * /*ivi_controller_surface*/,
    int32_t width, int32_t height) {
-   static_cast<struct surface *>(data)->parent->surface_configuration(
-      static_cast<struct surface *>(data)->id, width, height);
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_configuration(s->id, width, height);
 }
 
-void surface_orientation(void *data,
-                         struct ivi_controller_surface *ivi_controller_surface,
-                         int32_t orientation) {
-   static_cast<struct surface *>(data)->parent->surface_orientation(
-      static_cast<struct surface *>(data)->id, orientation);
+void surface_orientation(
+   void *data, struct ivi_controller_surface * /*ivi_controller_surface*/,
+   int32_t orientation) {
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_orientation(s->id, orientation);
 }
 
-void surface_pixelformat(void *data,
-                         struct ivi_controller_surface *ivi_controller_surface,
-                         int32_t pixelformat) {
-   static_cast<struct surface *>(data)->parent->surface_pixelformat(
-      static_cast<struct surface *>(data)->id, pixelformat);
+void surface_pixelformat(
+   void *data, struct ivi_controller_surface * /*ivi_controller_surface*/,
+   int32_t pixelformat) {
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_pixelformat(s->id, pixelformat);
 }
 
 void surface_layer(void *data,
-                   struct ivi_controller_surface *ivi_controller_surface,
+                   struct ivi_controller_surface * /*ivi_controller_surface*/,
                    struct ivi_controller_layer *layer) {
-   static_cast<struct surface *>(data)->parent->surface_layer(
-      static_cast<struct surface *>(data)->id, layer);
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_layer(s->id, layer);
 }
 
 void surface_stats(void *data,
-                   struct ivi_controller_surface *ivi_controller_surface,
+                   struct ivi_controller_surface * /*ivi_controller_surface*/,
                    uint32_t redraw_count, uint32_t frame_count,
                    uint32_t update_count, uint32_t pid,
                    const char *process_name) {
-   static_cast<struct surface *>(data)->parent->surface_stats(
-      static_cast<struct surface *>(data)->id, redraw_count, frame_count,
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_stats(s->id, redraw_count, frame_count,
       update_count, pid, process_name);
 }
 
-void surface_destroyed(void *data,
-                       struct ivi_controller_surface *ivi_controller_surface) {
-   static_cast<struct surface *>(data)->parent->surface_destroyed(
-      static_cast<struct surface *>(data)->id);
+void surface_destroyed(
+   void *data, struct ivi_controller_surface * /*ivi_controller_surface*/) {
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_destroyed(s->id);
 }
 
 void surface_content(void *data,
-                     struct ivi_controller_surface *ivi_controller_surface,
+                     struct ivi_controller_surface * /*ivi_controller_surface*/,
                      int32_t content_state) {
-   static_cast<struct surface *>(data)->parent->surface_content(
-      static_cast<struct surface *>(data)->id, content_state);
+   auto s = static_cast<struct surface *>(data);
+   s->parent->surface_content(s->id, content_state);
 }
 
 constexpr struct ivi_controller_surface_listener surface_listener = {
@@ -433,32 +547,81 @@ constexpr struct ivi_controller_surface_listener surface_listener = {
    surface_destroyed,
    surface_content,
 };
-}
+}  // namespace
 
 surface::surface(uint32_t i, struct controller *c)
    : wayland_proxy(ivi_controller_surface_create(c->proxy, i)),
-     controlled_entity(c, i) {
+     controller_child(c, i),
+     dst_rect{},
+     src_rect{},
+     size{},
+     orientation{},
+     visibility{},
+     opacity{1.f} {
+   this->parent->add_proxy_to_id_mapping(this->proxy, i);
    ivi_controller_surface_add_listener(this->proxy, &surface_listener, this);
 }
 
 surface::~surface() {
    logdebug("%s surface %i @ %p", __func__, this->id, this->proxy);
+   this->parent->remove_proxy_to_id_mapping(this->proxy);
    ivi_controller_surface_destroy(this->proxy, 1);
    this->proxy = nullptr;
 }
 
+void surface::set_visibility(uint32_t visibility) {
+   ivi_controller_surface_set_visibility(this->proxy, visibility);
+}
+
+void surface::set_opacity(wl_fixed_t opacity) {
+   ivi_controller_surface_set_opacity(this->proxy, opacity);
+}
+
+void surface::set_source_rectangle(int32_t x, int32_t y, int32_t width,
+                                   int32_t height) {
+   ivi_controller_surface_set_source_rectangle(this->proxy, x, y, width,
+                                               height);
+}
+
+void surface::set_destination_rectangle(int32_t x, int32_t y, int32_t width,
+                                        int32_t height) {
+   ivi_controller_surface_set_destination_rectangle(this->proxy, x, y, width,
+                                                    height);
+}
+
+void surface::set_configuration(int32_t width, int32_t height) {
+   ivi_controller_surface_set_configuration(this->proxy, width, height);
+}
+
+void surface::set_orientation(int32_t orientation) {
+   ivi_controller_surface_set_orientation(this->proxy, orientation);
+}
+
+void surface::screenshot(const char *filename) {
+   ivi_controller_surface_screenshot(this->proxy, filename);
+}
+
+void surface::send_stats() { ivi_controller_surface_send_stats(this->proxy); }
+
+void surface::destroy(int32_t destroy_scene_object) {
+   ivi_controller_surface_destroy(this->proxy, destroy_scene_object);
+}
+
 void controller::surface_visibility(uint32_t id, int32_t visibility) {
    logdebug("genivi::surface %s @ %p v %i", __func__, this->proxy, visibility);
+   this->surfaces[id]->visibility = visibility;
 }
 
 void controller::surface_opacity(uint32_t id, float opacity) {
    logdebug("genivi::surface %s @ %p o %f", __func__, this->proxy, opacity);
+   this->surfaces[id]->opacity = opacity;
 }
 
 void controller::surface_source_rectangle(uint32_t id, int32_t x, int32_t y,
                                           int32_t width, int32_t height) {
    logdebug("genivi::surface %s @ %p x %i y %i w %i h %i", __func__,
             this->proxy, x, y, width, height);
+   this->surfaces[id]->src_rect = rect{uint32_t(width), uint32_t(height), x, y};
 }
 
 void controller::surface_destination_rectangle(uint32_t id, int32_t x,
@@ -466,28 +629,32 @@ void controller::surface_destination_rectangle(uint32_t id, int32_t x,
                                                int32_t height) {
    logdebug("genivi::surface %s @ %p x %i y %i w %i h %i", __func__,
             this->proxy, x, y, width, height);
+   this->surfaces[id]->dst_rect = rect{uint32_t(width), uint32_t(height), x, y};
 }
 
 void controller::surface_configuration(uint32_t id, int32_t width,
                                        int32_t height) {
    logdebug("genivi::surface %s @ %p w %i h %i", __func__, this->proxy, width,
             height);
+   this->surfaces[id]->size = size{uint32_t(width), uint32_t(height)};
 }
 
 void controller::surface_orientation(uint32_t id, int32_t orientation) {
    logdebug("genivi::surface %s @ %p o %i", __func__, this->proxy, orientation);
+   this->surfaces[id]->orientation = orientation;
 }
 
-void controller::surface_pixelformat(uint32_t id, int32_t pixelformat) {
+void controller::surface_pixelformat(uint32_t /*id*/, int32_t pixelformat) {
    logdebug("genivi::surface %s @ %p f %i", __func__, this->proxy, pixelformat);
 }
 
-void controller::surface_layer(uint32_t id,
+void controller::surface_layer(uint32_t /*id*/,
                                struct ivi_controller_layer *layer) {
-   logdebug("genivi::surface %s @ %p l @ %p", __func__, this->proxy, layer);
+   logdebug("genivi::surface %s @ %p l %u @ %p", __func__, this->proxy,
+            this->layer_proxy_to_id[uintptr_t(layer)], layer);
 }
 
-void controller::surface_stats(uint32_t id, uint32_t redraw_count,
+void controller::surface_stats(uint32_t /*id*/, uint32_t redraw_count,
                                uint32_t frame_count, uint32_t update_count,
                                uint32_t pid, const char *process_name) {
    logdebug("genivi::surface %s @ %p r %u f %u u %u pid %u p %s", __func__,
@@ -497,11 +664,62 @@ void controller::surface_stats(uint32_t id, uint32_t redraw_count,
 
 void controller::surface_destroyed(uint32_t id) {
    logdebug("genivi::surface %s @ %p", __func__, this->proxy);
+   this->surfaces.erase(id);
 }
 
 void controller::surface_content(uint32_t id, int32_t content_state) {
    logdebug("genivi::surface %s @ %p s %i", __func__, this->proxy,
             content_state);
+   if (content_state == IVI_CONTROLLER_SURFACE_CONTENT_STATE_CONTENT_REMOVED) {
+      add_task("remove surface",
+               [id](struct controller *c) { c->surfaces.erase(id); });
+   }
+}
+
+void controller::add_proxy_to_id_mapping(struct ivi_controller_surface *p, uint32_t id) {
+   this->surface_proxy_to_id[uintptr_t(p)] = id;
+   logdebug("Add surface proxy mapping for %p (%u)", p, id);
+}
+
+void controller::remove_proxy_to_id_mapping(struct ivi_controller_surface *p) {
+   logdebug("Remove surface proxy mapping for %p", p);
+   this->surface_proxy_to_id.erase(uintptr_t(p));
+}
+
+void controller::add_proxy_to_id_mapping(struct ivi_controller_layer *p, uint32_t id) {
+   logdebug("Add layer proxy mapping for %p (%u)", p, id);
+   this->layer_proxy_to_id[uintptr_t(p)] = id;
+}
+
+void controller::remove_proxy_to_id_mapping(struct ivi_controller_layer *p) {
+   logdebug("Remove layer proxy mapping for %p", p);
+   this->layer_proxy_to_id.erase(uintptr_t(p));
+}
+
+void controller::add_proxy_to_id_mapping(struct wl_output *p, uint32_t id) {
+   logdebug("Add screen proxy mapping for %p (%u)", p, id);
+   this->screen_proxy_to_id[uintptr_t(p)] = id;
+}
+
+void controller::remove_proxy_to_id_mapping(struct wl_output *p) {
+   logdebug("Remove screen proxy mapping for %p", p);
+   this->screen_proxy_to_id.erase(uintptr_t(p));
+}
+
+void controller::add_task(char const *name, std::function<void(struct controller *)> &&f) {
+   this->pending.emplace_back(std::make_pair(name, f));
+}
+
+void controller::execute_pending() {
+   if (!this->pending.empty()) {
+      for (auto &t : this->pending) {
+         logdebug("executing task '%s'", t.first);
+         t.second(this);
+      }
+      this->pending.clear();
+      ivi_controller_commit_changes(this->proxy);
+      // XXX: No flush here...
+   }
 }
 
 //
@@ -512,5 +730,22 @@ void controller::surface_content(uint32_t id, int32_t content_state) {
 //
 screen::screen(uint32_t i, struct controller *c,
                struct ivi_controller_screen *p)
-   : wayland_proxy(p), controlled_entity(c, i) {}
+   : wayland_proxy(p), controller_child(c, i) {
+   logdebug("genivi::screen @ %p id %u", p, i);
 }
+
+void screen::clear() { ivi_controller_screen_clear(this->proxy); }
+
+void screen::add_layer(layer *l) {
+   ivi_controller_screen_add_layer(this->proxy, l->proxy);
+}
+
+void screen::set_render_order(std::vector<uint32_t> const &ro) {
+   struct wl_array wlro {
+      .size = ro.size() * sizeof(ro[0]), .alloc = ro.capacity() * sizeof(ro[0]),
+      .data = const_cast<void *>(static_cast<void const *>(ro.data()))
+   };
+   ivi_controller_screen_set_render_order(this->proxy, &wlro);
+}
+
+}  // namespace genivi