App: be a little more thorough when activating a surface
[staging/windowmanager.git] / src / app.cpp
index 6a7f5b4..f76ebbd 100644 (file)
  * limitations under the License.
  */
 
-//
-// Created by mfritzsc on 7/11/17.
-//
-
 #include "app.hpp"
 #include "json_helper.hpp"
 #include "layers.hpp"
 
 #include <json-c/json.h>
 
+#include <algorithm>
 #include <bits/signum.h>
 #include <csignal>
 #include <fstream>
+#include <thread>
 #include <json.hpp>
 
 namespace wm {
@@ -45,22 +43,19 @@ App *g_app;
 using json = nlohmann::json;
 
 struct wm::area area_from_json(json const &j) {
-   DB(j);
    return wm::area{
-      j["name"].get<std::string>(),
+      j["name"],
       {
-         get<int32_t>(j["width"]), get<int32_t>(j["height"]),
-         get<int32_t>(j["x"]), get<int32_t>(j["y"]),
+         j["width"], j["height"], j["x"], j["y"],
       },
-      get<uint32_t>(j["zorder"]),
+      j["zorder"],
    };
 }
 
 result<struct layout> layout_from_json(json const &j) {
-   DB(j);
    auto &ja = j["areas"];
 
-   auto l = layout{j["name"].get<std::string>(), uint32_t(ja.size()), {}};
+   auto l = layout{j["name"], uint32_t(ja.size()), {}};
 
    if (ja.size() > layout::MAX_N_AREAS) {
       return Err<struct layout>("Invalid number of areas in layout");
@@ -87,7 +82,7 @@ result<json> file_to_json(char const *filename) {
 
 // Will throw if parsing fails
 struct result<layouts_type> load_layout(char const *filename) {
-   DB("loading layout from " << filename);
+   logdebug("loading layout from %s", filename);
 
    auto j = file_to_json(filename);
    if (j.is_err()) {
@@ -105,7 +100,7 @@ struct result<layouts_type> load_layout(char const *filename) {
 
 struct result<layer_map>
    load_layer_map(char const *filename) {
-   DB("loading IDs from " << filename);
+   logdebug("loading IDs from %s", filename);
 
    auto j = file_to_json(filename);
    if (j.is_err()) {
@@ -132,14 +127,15 @@ App::App(wl::display *d)
      outputs(),
      config(),
      layouts(),
-     layers() {
+     layers(),
+     id_alloc{} {
    assert(g_app == nullptr);
    g_app = this;
 
    try {
       {
          auto l = load_layer_map(
-                 this->config.get_string("layers.json").value().c_str());
+            this->config.get_string("layers.json").value().c_str());
          if (l.is_ok()) {
             this->layers = l.unwrap();
          } else {
@@ -148,8 +144,8 @@ App::App(wl::display *d)
       }
 
       {
-         auto l = load_layout(
-                 this->config.get_string("layout.json").value().c_str());
+         auto l =
+            load_layout(this->config.get_string("layout.json").value().c_str());
          if (l.is_ok()) {
             this->layouts = l.unwrap();
          } else {
@@ -180,7 +176,7 @@ int App::init() {
 
    this->display->add_global_handler(
       "ivi_controller", [this](wl_registry *r, uint32_t name, uint32_t v) {
-         this->controller = std::make_unique<genivi::controller>(r, name, v);
+         this->controller = std::make_unique<struct genivi::controller>(r, name, v);
 
          // Init controller hooks
          this->controller->chooks = &this->chooks;
@@ -200,7 +196,7 @@ int App::init() {
    // Third level objects
    this->display->roundtrip();
 
-   return init_layout();
+   return init_layers();
 }
 
 int App::dispatch_events() {
@@ -213,7 +209,7 @@ int App::dispatch_events() {
    this->display->flush();
 
    // execute pending tasks, that is layout changes etc.
-   this->execute_pending();
+   //this->execute_pending();
 
    return 0;
 }
@@ -224,7 +220,7 @@ int App::dispatch_events() {
 // | | | | | | |_    | | (_| | |_| | (_) | |_| | |_| |  | |
 // |_|_| |_|_|\__|___|_|\__,_|\__, |\___/ \__,_|\__| |  | |
 //              |_____|       |___/                 \_\/_/
-int App::init_layout() {
+int App::init_layers() {
    if (!this->controller) {
       logerror("ivi_controller global not available");
       return -1;
@@ -272,22 +268,52 @@ int App::init_layout() {
    return 0;
 }
 
-void App::surface_set_layout(uint32_t surface_id) {
-   auto o_layer_id = this->layers.get_layer_id(surface_id);
+namespace {
 
-   if (!o_layer_id) {
-      logerror("Surface %d is not associated with any layer!", int(surface_id));
-      return;
+// This can fix the HomeScreen...
+void redraw_fix(App *app, std::unique_ptr<genivi::surface> &s, int x, int y, int w, int h) {
+   { // XXX: Work around weston redraw issues
+      // trigger an update by changing the source dimensions!
+      s->set_configuration(w + 1, h);
+      s->set_source_rectangle(0, 0, w + 1, h);
+      s->set_destination_rectangle(x, y, w + 1, h);
+      app->controller->commit_changes();
+      app->display->roundtrip();
+
+      // wait some time, for the process to do its thing...
+      using namespace std::chrono_literals;
+      std::this_thread::sleep_for(100ms);
+
+      // Set a different size then what we actually want.
+      s->set_configuration(w, h);
+      s->set_source_rectangle(0, 0, w, h);
+      s->set_destination_rectangle(x, y, w, h);
+      app->controller->commit_changes();
+      app->display->roundtrip();
    }
+}
+
+}  // namespace
 
+void App::surface_init_layout(uint32_t surface_id) {
    if (!this->controller->surface_exists(surface_id)) {
       logerror("Surface %d does not exist", int(surface_id));
       return;
    }
 
+   auto o_layer_id = this->layers.get_layer_id(surface_id);
+
+   if (!o_layer_id) {
+      logerror("Surface %d is not associated with any layer!", int(surface_id));
+      return;
+   }
+
    uint32_t layer_id = o_layer_id.value();
+   logdebug("surface_set_layout for surface %u on layer %u", surface_id,
+            layer_id);
 
-   auto rect = this->layers.get_layer_rect(surface_id).value();
+   auto const &layer = this->layers.get_layer(layer_id);
+   auto rect = layer.value().rect;
    auto &s = this->controller->surfaces[surface_id];
 
    int x = rect.x;
@@ -303,38 +329,100 @@ void App::surface_set_layout(uint32_t surface_id) {
    if (h < 0) {
       h = this->controller->output_size.h + 1 + h;
    }
-   logdebug("Computed rect={ %d, %d, %d, %d }", x, y, w, h);
 
    // configure surface to wxh dimensions
    s->set_configuration(w, h);
-   // set source rect to "entire surface"
+
+   // set source reactangle, even if we should not need to set it.
    s->set_source_rectangle(0, 0, w, h);
+
    // set destination to the display rectangle
    s->set_destination_rectangle(x, y, w, h);
 
-   // XXX: visibility should be determined independently of our
-   //      layer + geometry setup.
-   s->set_visibility(1);
+   s->set_visibility(0);
+   s->set_opacity(256);
+
+   this->controller->commit_changes();
+   this->display->roundtrip();
+
+   redraw_fix(this, s, x, y, w, h);
+
    this->controller->layers[layer_id]->add_surface(s.get());
 
+   // activate the main_surface right away
+   if (surface_id == static_cast<unsigned>(this->layers.main_surface)) {
+      logdebug("Activating main_surface (%d)", surface_id);
+
+      this->activate_surface(this->lookup_name(surface_id).value_or("unknown-name").c_str());
+   }
+
    logdebug("Surface %u now on layer %u with rect { %d, %d, %d, %d }",
             surface_id, layer_id, x, y, w, h);
 }
 
-void App::add_task(char const *name, std::function<void()> &&f) {
-   this->pending.emplace_back(std::make_pair(name, f));
+char const *App::activate_surface(char const *drawing_name) {
+   int surface_id = -1;
+
+   {
+      auto oid = this->lookup_id(drawing_name);
+      if (oid) {
+         surface_id = oid.value();
+      } else {
+         return "Surface does not exist";
+      }
+   }
+
+   if (!this->controller->surface_exists(surface_id)) {
+      return "Surface does not exist";
+   }
+
+   // This should involve a policy check, but as we do not (yet) have
+   // such a thing, we will just switch to this surface.
+   // XXX: input focus missing!!1
+
+   // Make it visible, no (or little effect) if already visible
+   auto &s = this->controller->surfaces[surface_id];
+
+   // Set all others invisible
+   for (auto &i : this->controller->surfaces) {
+      auto &si = this->controller->sprops[i.second->id];
+      if (si.visibility != 0 &&
+          int(si.id) != this->layers.main_surface) {
+         this->deactivate(i.second->id);
+      }
+   }
+   this->activate(s->id);
+
+   // commit changes
+   this->controller->commit_changes();
+   this->display->flush();
+
+   // no error
+   return nullptr;
 }
 
-void App::execute_pending() {
-   if (!this->pending.empty()) {
-      for (auto &t : this->pending) {
-         logdebug("executing task '%s'", t.first);
-         t.second();
+char const *App::deactivate_surface(char const *drawing_name) {
+   int surface_id = -1;
+
+   {
+      auto oid = this->lookup_id(drawing_name);
+      if (oid) {
+         surface_id = oid.value();
+      } else {
+         return "Surface does not exist";
       }
-      this->pending.clear();
-      this->controller->commit_changes();
-      this->display->flush();
    }
+
+   if (surface_id == this->layers.main_surface) {
+      return "Cannot deactivate main_surface";
+   }
+
+   this->deactivate(surface_id);
+
+   this->controller->commit_changes();
+   this->display->flush();
+
+   return nullptr;
 }
 
 //                      _          _   _____                 _
@@ -344,16 +432,79 @@ void App::execute_pending() {
 // | .__/|_|  \___/_/\_\_|\___|\__,_| |_____| \_/ \___|_| |_|\__|___/
 // |_|
 void App::surface_created(uint32_t surface_id) {
-   DB("surface_id is " << surface_id);
+   logdebug("surface_id is %u", surface_id);
 
-   // We need to execute the surface setup after its creation.
-   // XXX: perhaps move the late-tasks functionality to App?
-   this->add_task("surface_set_layout",
-                  [surface_id, this] { this->surface_set_layout(surface_id); });
+   this->surface_init_layout(surface_id);
 }
 
 void App::surface_removed(uint32_t surface_id) {
-   DB("surface_id is " << surface_id);
+   logdebug("surface_id is %u", surface_id);
+
+   this->id_alloc.remove_id(surface_id);
+}
+
+void App::emit_activated(char const *label) {
+   this->api.send_event("activated", label);
+}
+
+void App::emit_deactivated(char const *label) {
+   this->api.send_event("deactivated", label);
+}
+
+void App::emit_syncdraw(char const *label) {
+   this->api.send_event("syncdraw", label);
+}
+
+void App::emit_flushdraw(char const *label) {
+   this->api.send_event("syncdraw", label);
+}
+
+void App::emit_visible(char const *label, bool is_visible) {
+   this->api.send_event(is_visible ? "visible" : "invisible", label);
+}
+
+result<int> App::request_surface(char const *drawing_name) {
+   auto lid = this->layers.get_layer_id(std::string(drawing_name));
+   if (!lid) {
+      // XXX: to we need to put these applications on the App layer?
+      return Err<int>("Drawing name does not match any role");
+   }
+
+   auto rname = this->lookup_id(drawing_name);
+   if (!rname) {
+      // name does not exist yet, allocate surface id...
+      auto id = int(this->id_alloc.generate_id(drawing_name));
+      this->layers.add_surface(id, lid.value());
+
+      // XXX: we set the main_surface[_name] here and now,
+      // not sure if we want this, but it worked so far.
+      if (!this->layers.main_surface_name.empty() &&
+          this->layers.main_surface_name == drawing_name) {
+         this->layers.main_surface = id;
+         logdebug("Set main_surface id to %u", id);
+      }
+
+      return Ok<int>(id);
+   }
+
+   // Check currently registered drawing names if it is already there.
+   return Err<int>("Surface already present");
+}
+
+void App::activate(unsigned id) {
+   if (this->controller->sprops[id].visibility == 0) {
+      this->controller->surfaces[id]->set_visibility(1);
+      this->emit_activated(
+              this->lookup_name(id).value_or("unknown-name").c_str());
+   }
+}
+
+void App::deactivate(unsigned id) {
+   if (this->controller->sprops[id].visibility != 0) {
+      this->controller->surfaces[id]->set_visibility(0);
+      this->emit_deactivated(
+              this->lookup_name(id).value_or("unknown-name").c_str());
+   }
 }
 
 //  _     _           _ _                            _   _                 _
@@ -362,18 +513,45 @@ void App::surface_removed(uint32_t surface_id) {
 // | |_) | | | | | (_| | | | | | (_| |  | (_| | |_) | | | | | | | | | |_) | |
 // |_.__/|_|_| |_|\__,_|_|_| |_|\__, |___\__,_| .__/|_| |_|_| |_| |_| .__/|_|
 //                              |___/_____|   |_|                   |_|
-binding_api::result_type binding_api::register_surface(uint32_t appid,
-                                                       uint32_t surfid) {
-   logdebug("%s appid %u surfid %u", __func__, appid, surfid);
-   if (appid > 0xff) {
-      return Err<json_object *>("invalid appid");
+binding_api::result_type binding_api::request_surface(
+   char const *drawing_name) {
+   auto r = this->app->request_surface(drawing_name);
+   if (r.is_err()) {
+      return Err<json_object *>(r.unwrap_err());
    }
+   return Ok(json_object_new_int(r.unwrap()));
+}
 
-   if (surfid > 0xffff) {
-      return Err<json_object *>("invalid surfaceid");
+binding_api::result_type binding_api::activate_surface(
+   char const *drawing_name) {
+   logdebug("%s drawing_name %s", __func__, drawing_name);
+   auto r = this->app->activate_surface(drawing_name);
+   if (r != nullptr) {
+      logdebug("%s failed with error: %s", __func__, r);
+      return Err<json_object *>(r);
    }
+   return Ok(json_object_new_object());
+}
+
+binding_api::result_type binding_api::deactivate_surface(char const* drawing_name) {
+   logdebug("%s drawing_name %s", __func__, drawing_name);
+   auto r = this->app->deactivate_surface(drawing_name);
+   if (r != nullptr) {
+      logdebug("%s failed with error: %s", __func__, r);
+      return Err<json_object *>(r);
+   }
+   return Ok(json_object_new_object());
+}
 
-   return Ok(json_object_new_int((appid << 16) + surfid));
+binding_api::result_type binding_api::enddraw(char const* drawing_name) {
+   logdebug("%s drawing_name %s", __func__, drawing_name);
+   return Err<json_object*>("not implemented");
+}
+
+binding_api::result_type binding_api::list_drawing_names() {
+   logdebug("%s", __func__);
+   json j = this->app->id_alloc.name2id;
+   return Ok(json_tokener_parse(j.dump().c_str()));
 }
 
 binding_api::result_type binding_api::debug_layers() {
@@ -416,8 +594,4 @@ void controller_hooks::surface_removed(uint32_t surface_id) {
    this->app->surface_removed(surface_id);
 }
 
-void controller_hooks::add_task(char const *name, std::function<void()> &&f) {
-   this->app->add_task(name, std::move(f));
-}
-
 }  // namespace wm