From: Marcus Fritzsch <marcus_fritzsch@mentor.com>
Date: Tue, 5 Sep 2017 10:12:51 +0000 (+0200)
Subject: App: be a little more thorough when activating a surface
X-Git-Tag: 4.99.1~131
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?p=staging%2Fwindowmanager.git;a=commitdiff_plain;h=76fb6aef7ca4cf8f922a1076eaafef3c19e3fd07

App: be a little more thorough when activating a surface

Signed-off-by: Marcus Fritzsch <marcus_fritzsch@mentor.com>
---

diff --git a/src/app.cpp b/src/app.cpp
index 26dc5f6..f76ebbd 100644
--- a/src/app.cpp
+++ b/src/app.cpp
@@ -386,7 +386,8 @@ char const *App::activate_surface(char const *drawing_name) {
    // 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) {
+      if (si.visibility != 0 &&
+          int(si.id) != this->layers.main_surface) {
          this->deactivate(i.second->id);
       }
    }
@@ -526,6 +527,7 @@ binding_api::result_type binding_api::activate_surface(
    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());
@@ -535,6 +537,7 @@ binding_api::result_type binding_api::deactivate_surface(char const* drawing_nam
    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());