diff options
author | Marcus Fritzsch <marcus_fritzsch@mentor.com> | 2017-08-31 17:27:02 +0200 |
---|---|---|
committer | Marcus Fritzsch <marcus_fritzsch@mentor.com> | 2017-09-04 16:54:02 +0200 |
commit | 4a67c02ff81303b9e95537cbbaddf422d99f2fbd (patch) | |
tree | 5b500be392b1501b118d81e1b04dfbf4be615251 /src | |
parent | 74826b4760a3946e3f196a00c6a9f292a810f855 (diff) |
app: remove previous work around experiments, add one that fixes HomeScreen
Signed-off-by: Marcus Fritzsch <marcus_fritzsch@mentor.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/app.cpp | 65 |
1 files changed, 44 insertions, 21 deletions
diff --git a/src/app.cpp b/src/app.cpp index 7ff353f..6d47928 100644 --- a/src/app.cpp +++ b/src/app.cpp @@ -270,6 +270,35 @@ int App::init_layout() { return 0; } +namespace { + +// 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, h); + s->set_source_rectangle(0, 0, w, h); + s->set_destination_rectangle(x, y, w, 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 + 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(); + + std::this_thread::sleep_for(100ms); + } +} + +} // namespace + void App::surface_set_layout(uint32_t surface_id) { if (!this->controller->surface_exists(surface_id)) { logerror("Surface %d does not exist", int(surface_id)); @@ -305,20 +334,29 @@ void App::surface_set_layout(uint32_t surface_id) { h = this->controller->output_size.h + 1 + h; } + redraw_fix(this, s, x, y, w, h); + // configure surface to wxh dimensions s->set_configuration(w, h); + // 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: The main_surface will be visible regardless - //s->set_visibility( - // surface_id == static_cast<unsigned>(this->layers.main_surface) ? 1 : 0); + s->set_visibility(0); + s->set_opacity(256); + + this->controller->commit_changes(); + this->display->roundtrip(); + 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(surface_id); } @@ -341,8 +379,10 @@ char const *App::activate_surface(uint32_t surface_id) { // Set all others invisible for (auto &i : this->controller->surfaces) { auto &si = this->controller->sprops[i.second->id]; - if (si.id != s->id && int(si.id) != this->layers.main_surface) { + if (int(si.id) != this->layers.main_surface) { i.second->set_visibility(0); + this->controller->commit_changes(); + this->display->flush(); } } s->set_visibility(1); @@ -351,23 +391,6 @@ char const *App::activate_surface(uint32_t surface_id) { this->controller->commit_changes(); this->display->flush(); - auto set_vis = [&s, this](int id, int vis) { - using namespace std::chrono_literals; - - std::this_thread::sleep_for(200ms); - char *buf; - asprintf(&buf, "LayerManagerControl set surface %d visibility %d", id, vis); - //system("LayerManagerControl get scene"); - //system("sleep 0.1"); - system(buf); - - //s->set_visibility(vis); - //this->controller->commit_changes(); - //this->display->roundtrip(); - }; - set_vis(s->id, 0); - set_vis(s->id, 1); - // Current active surface is the first in last_active this->last_active.push_front(s->id); /// // XXX: I am not sure I even need 5 elements... |