diff options
author | Joel Winarske <joel.winarske@gmail.com> | 2024-06-28 13:13:20 -0700 |
---|---|---|
committer | Jan-Simon Moeller <jsmoeller@linuxfoundation.org> | 2024-07-09 09:34:09 +0000 |
commit | 1696e85d2d81cc42532bc3968eb8db8c979e59fb (patch) | |
tree | 5f6205f1a8fc7dba73a402a3572ba3e81571d183 | |
parent | 8933d6c6b94f3b395167456ea9f8917a4ecb3bd4 (diff) |
flutter-auto v2
-remove patches as upstream has picked them up
-update desktop-auto.json to v2.0 baseline
-mask libcamera log message when running flutter-auto.
flutter-auto handles libcamera messages directly.
-add xdg-shell-app-id to service
Bug-AGL: SPEC-5193
Change-Id: Idf054a921b055774262ea0c760ec134ba70d8128
Signed-off-by: Joel Winarske <joel.winarske@gmail.com>
Reviewed-on: https://gerrit.automotivelinux.org/gerrit/c/AGL/meta-agl/+/30048
ci-image-boot-test: Jenkins Job builder account
ci-image-build: Jenkins Job builder account
Reviewed-by: Marius Vlad <marius.vlad@collabora.com>
Tested-by: Jenkins Job builder account
Reviewed-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
(cherry picked from commit 8d6e6f806f20c7848565e52132791ac14091472b)
Reviewed-on: https://gerrit.automotivelinux.org/gerrit/c/AGL/meta-agl/+/30123
10 files changed, 19 insertions, 863 deletions
diff --git a/meta-agl-flutter/dynamic-layers/flutter-apps-layer/recipes-graphics/flutter-apps/first-party/flutter-gallery/flutter-gallery.service b/meta-agl-flutter/dynamic-layers/flutter-apps-layer/recipes-graphics/flutter-apps/first-party/flutter-gallery/flutter-gallery.service index 28b967866..0b29c3f82 100644 --- a/meta-agl-flutter/dynamic-layers/flutter-apps-layer/recipes-graphics/flutter-apps/first-party/flutter-gallery/flutter-gallery.service +++ b/meta-agl-flutter/dynamic-layers/flutter-apps-layer/recipes-graphics/flutter-apps/first-party/flutter-gallery/flutter-gallery.service @@ -7,7 +7,7 @@ User=agl-driver EnvironmentFile=/etc/default/flutter EnvironmentFile=-/etc/default/flutter-gallery Environment=XDG_RUNTIME_DIR=/run/user/1001/ -ExecStart=/usr/bin/flutter-auto --b=/usr/share/flutter/flutter-gallery/${FLUTTER_VERSION}/${FLUTTER_RUNTIME} --f --window-type=BG +ExecStart=/usr/bin/flutter-auto --b=/usr/share/flutter/flutter-gallery/${FLUTTER_VERSION}/${FLUTTER_RUNTIME} --f --window-type=BG --xdg-shell-app-id=flutter-gallery [Install] WantedBy=graphical.target diff --git a/meta-agl-flutter/recipes-config/agl-flutter-env/agl-flutter-env_1.0.bb b/meta-agl-flutter/recipes-config/agl-flutter-env/agl-flutter-env_1.0.bb index aaff166bf..80f7fb3f1 100644 --- a/meta-agl-flutter/recipes-config/agl-flutter-env/agl-flutter-env_1.0.bb +++ b/meta-agl-flutter/recipes-config/agl-flutter-env/agl-flutter-env_1.0.bb @@ -12,4 +12,5 @@ do_install() { install -d ${D}${sysconfdir}/default echo "FLUTTER_VERSION=${FLUTTER_SDK_TAG}" > ${D}${sysconfdir}/default/flutter echo "FLUTTER_RUNTIME=${AGL_FLUTTER_DEFAULT_RUNTIME}" >> ${D}${sysconfdir}/default/flutter + echo "LIBCAMERA_LOG_LEVELS=*:ERROR" >> ${D}${sysconfdir}/default/flutter } diff --git a/meta-agl-flutter/recipes-graphics/toyota/files/0001-activation_area-Add-missing-width-and-height.patch b/meta-agl-flutter/recipes-graphics/toyota/files/0001-activation_area-Add-missing-width-and-height.patch deleted file mode 100644 index fb3a5b3f7..000000000 --- a/meta-agl-flutter/recipes-graphics/toyota/files/0001-activation_area-Add-missing-width-and-height.patch +++ /dev/null @@ -1,190 +0,0 @@ -From 2c55093cda17dd16d98a8fb0d26717b29d5d0e75 Mon Sep 17 00:00:00 2001 -From: Marius Vlad <marius.vlad@collabora.com> -Date: Thu, 28 Dec 2023 19:20:31 +0200 -Subject: [PATCH 1/2] activation_area: Add missing width and height - -Setting up an activation area can also require a different rectangle -size, so allow to set-up one. - -Note that there's no actual check if the values are invalid, for -instance like exceeding the output's dimensions. - -Bug-AGL: SPEC-5038 -Signed-off-by: Marius Vlad <marius.vlad@collabora.com> ---- - shell/configuration/configuration.cc | 8 ++++++++ - shell/configuration/configuration.h | 2 ++ - shell/view/flutter_view.cc | 3 ++- - shell/wayland/display.cc | 10 +++++----- - shell/wayland/display.h | 8 +++++++- - shell/wayland/window.cc | 13 +++++++++---- - shell/wayland/window.h | 4 ++++ - 7 files changed, 37 insertions(+), 11 deletions(-) - -diff --git a/shell/configuration/configuration.cc b/shell/configuration/configuration.cc -index 2cec924..63b7a82 100644 ---- a/shell/configuration/configuration.cc -+++ b/shell/configuration/configuration.cc -@@ -104,9 +104,17 @@ void Configuration::getViewParameters( - - instance.view.activation_area_x = static_cast<uint32_t>(val["x"].GetInt()); - instance.view.activation_area_y = static_cast<uint32_t>(val["y"].GetInt()); -+ instance.view.activation_area_width = -+ static_cast<uint32_t>(val["width"].GetInt()); -+ instance.view.activation_area_height = -+ static_cast<uint32_t>(val["height"].GetInt()); - - SPDLOG_DEBUG("activation area x {}", instance.view.activation_area_x); - SPDLOG_DEBUG("activation area y {}", instance.view.activation_area_y); -+ SPDLOG_DEBUG("activation area width {}", -+ instance.view.activation_area_width); -+ SPDLOG_DEBUG("activation area height {}", -+ instance.view.activation_area_height); - } - } - -diff --git a/shell/configuration/configuration.h b/shell/configuration/configuration.h -index 0e3d721..b0ccbc9 100644 ---- a/shell/configuration/configuration.h -+++ b/shell/configuration/configuration.h -@@ -44,6 +44,8 @@ class Configuration { - uint32_t height; - uint32_t activation_area_x; - uint32_t activation_area_y; -+ uint32_t activation_area_width; -+ uint32_t activation_area_height; - bool fullscreen; - bool fullscreen_set; - double pixel_ratio; -diff --git a/shell/view/flutter_view.cc b/shell/view/flutter_view.cc -index ca51f4e..7b48b85 100644 ---- a/shell/view/flutter_view.cc -+++ b/shell/view/flutter_view.cc -@@ -67,9 +67,10 @@ FlutterView::FlutterView(Configuration::Config config, - m_wayland_window = std::make_shared<WaylandWindow>( - m_index, display, m_config.view.window_type, - m_wayland_display->GetWlOutput(m_config.view.wl_output_index), -- m_config.view.wl_output_index, m_config.app_id, m_config.view.fullscreen, -+ m_config.view.wl_output_index, m_config.app_id, m_config.view.fullscreen, - m_config.view.width, m_config.view.height, m_config.view.pixel_ratio, - m_config.view.activation_area_x, m_config.view.activation_area_y, -+ m_config.view.activation_area_width, m_config.view.activation_area_height, - m_backend.get(), m_config.view.ivi_surface_id); - } - -diff --git a/shell/wayland/display.cc b/shell/wayland/display.cc -index aba050a..0b97954 100644 ---- a/shell/wayland/display.cc -+++ b/shell/wayland/display.cc -@@ -857,16 +857,16 @@ void Display::AglShellDoReady() const { - - void Display::AglShellDoSetupActivationArea(uint32_t x, - uint32_t y, -+ uint32_t width, -+ uint32_t height, - uint32_t index) { -- uint32_t width = m_all_outputs[index]->width; -- uint32_t height = m_all_outputs[index]->height - (2 * y); -- - if (!m_agl.shell) - return; - - if (m_all_outputs[index]->transform == WL_OUTPUT_TRANSFORM_90) { -- width = m_all_outputs[index]->height; -- height = m_all_outputs[index]->width - (2 * y); -+ uint32_t tmp_width = width; -+ width = height; -+ height = tmp_width; - } - - SPDLOG_DEBUG("Using custom rectangle [{}x{}+{}x{}] for activation", width, -diff --git a/shell/wayland/display.h b/shell/wayland/display.h -index b919047..a792ad7 100644 ---- a/shell/wayland/display.h -+++ b/shell/wayland/display.h -@@ -172,6 +172,8 @@ class Display { - * @return void - * @param[in] x the x position for the activation rectangle - * @param[in] y the y position for the activation rectangle -+ * @param[in] width the width position for the activation rectangle -+ * @param[in] height the height position for the activation rectangle - * @param[index] the output, as a number - * @relation - * -@@ -193,7 +195,11 @@ class Display { - * | | - * -------------------- - */ -- void AglShellDoSetupActivationArea(uint32_t x, uint32_t y, uint32_t index); -+ void AglShellDoSetupActivationArea(uint32_t x, -+ uint32_t y, -+ uint32_t width, -+ uint32_t height, -+ uint32_t index); - - /** - * @brief Set Engine -diff --git a/shell/wayland/window.cc b/shell/wayland/window.cc -index 373b0a2..f816e58 100644 ---- a/shell/wayland/window.cc -+++ b/shell/wayland/window.cc -@@ -31,6 +31,8 @@ WaylandWindow::WaylandWindow(size_t index, - double pixel_ratio, - uint32_t activation_area_x, - uint32_t activation_area_y, -+ uint32_t activation_area_width, -+ uint32_t activation_area_height, - Backend* backend, - uint32_t ivi_surface_id) - : m_index(index), -@@ -42,7 +44,8 @@ WaylandWindow::WaylandWindow(size_t index, - m_geometry({width, height}), - m_window_size({width, height}), - m_pixel_ratio(pixel_ratio), -- m_activation_area({activation_area_x, activation_area_y}), -+ m_activation_area({activation_area_x, activation_area_y, -+ activation_area_width, activation_area_height}), - m_type(get_window_type(type)), - m_app_id(std::move(app_id)), - m_ivi_surface_id(ivi_surface_id), -@@ -98,10 +101,12 @@ WaylandWindow::WaylandWindow(size_t index, - case WINDOW_BG: - m_display->AglShellDoBackground(m_base_surface, 0); - if (m_activation_area.x > 0 && m_activation_area.y > 0) -- m_display->AglShellDoSetupActivationArea(m_activation_area.x, -- m_activation_area.y, 0); -+ m_display->AglShellDoSetupActivationArea( -+ m_activation_area.x, m_activation_area.y, m_activation_area.width, -+ m_activation_area.height, 0); - else -- m_display->AglShellDoSetupActivationArea(0, 160, 0); -+ m_display->AglShellDoSetupActivationArea(0, 160, m_activation_area.width, -+ m_activation_area.height, 0); - break; - case WINDOW_PANEL_TOP: - m_display->AglShellDoPanel(m_base_surface, AGL_SHELL_EDGE_TOP, 0); -diff --git a/shell/wayland/window.h b/shell/wayland/window.h -index 998a1c8..f66f70a 100644 ---- a/shell/wayland/window.h -+++ b/shell/wayland/window.h -@@ -67,6 +67,8 @@ class WaylandWindow { - double pixel_ratio, - uint32_t activation_area_x, - uint32_t activation_area_y, -+ uint32_t activation_area_width, -+ uint32_t activation_area_height, - Backend* backend, - uint32_t ivi_surface_id); - -@@ -166,6 +168,8 @@ class WaylandWindow { - struct { - uint32_t x; - uint32_t y; -+ uint32_t width; -+ uint32_t height; - } m_activation_area; - struct { - int32_t width; --- -2.35.1 - diff --git a/meta-agl-flutter/recipes-graphics/toyota/files/0001-display-Add-support-for-wl_output-version-4.patch b/meta-agl-flutter/recipes-graphics/toyota/files/0001-display-Add-support-for-wl_output-version-4.patch deleted file mode 100644 index 6b1b3e4c9..000000000 --- a/meta-agl-flutter/recipes-graphics/toyota/files/0001-display-Add-support-for-wl_output-version-4.patch +++ /dev/null @@ -1,117 +0,0 @@ -From 10d1d855a0ce4557cb710e73e3e7c9ab0dd0e734 Mon Sep 17 00:00:00 2001 -From: Marius Vlad <marius.vlad@collabora.com> -Date: Mon, 4 Dec 2023 14:16:36 +0200 -Subject: [PATCH 1/2] display: Add support for wl_output version 4 - -This allows support for wl_output.name and wl_output.desc be sent out -by the compositor that supports it. - -Signed-off-by: Marius Vlad <marius.vlad@collabora.com> ---- - shell/wayland/display.cc | 34 ++++++++++++++++++++++++++++++---- - shell/wayland/display.h | 28 ++++++++++++++++++++++++++++ - 2 files changed, 58 insertions(+), 4 deletions(-) - -diff --git a/shell/wayland/display.cc b/shell/wayland/display.cc -index 8e309ef..3ee814a 100644 ---- a/shell/wayland/display.cc -+++ b/shell/wayland/display.cc -@@ -191,9 +191,16 @@ void Display::registry_handle_global(void* data, - auto oi = std::make_shared<output_info_t>(); - std::fill_n(oi.get(), 1, output_info_t{}); - oi->global_id = name; -- oi->output = static_cast<struct wl_output*>( -- wl_registry_bind(registry, name, &wl_output_interface, -- std::min(static_cast<uint32_t>(2), version))); -+ // be compat with v2 as well -+ if (version >= WL_OUTPUT_NAME_SINCE_VERSION && -+ version >= WL_OUTPUT_DESCRIPTION_SINCE_VERSION) -+ oi->output = static_cast<struct wl_output*>( -+ wl_registry_bind(registry, name, &wl_output_interface, -+ std::min(static_cast<uint32_t>(4), version))); -+ else -+ oi->output = static_cast<struct wl_output*>( -+ wl_registry_bind(registry, name, &wl_output_interface, -+ std::min(static_cast<uint32_t>(2), version))); - wl_output_add_listener(oi->output, &output_listener, oi.get()); - SPDLOG_DEBUG("Wayland: Output [{}]", d->m_all_outputs.size()); - d->m_all_outputs.push_back(oi); -@@ -299,9 +306,28 @@ void Display::display_handle_done(void* data, - oi->done = true; - } - -+void Display::display_handle_name(void* data, -+ struct wl_output* /* wl_output */, -+ const char* name) { -+ auto* oi = static_cast<output_info_t*>(data); -+ oi->name = std::string(name); -+} -+ -+void Display::display_handle_desc(void* data, -+ struct wl_output* /* wl_output */, -+ const char* desc) { -+ auto* oi = static_cast<output_info_t*>(data); -+ oi->desc = std::string(desc); -+} -+ - const struct wl_output_listener Display::output_listener = { - display_handle_geometry, display_handle_mode, display_handle_done, -- display_handle_scale}; -+ display_handle_scale -+#if defined(WL_OUTPUT_NAME_SINCE_VERSION) && \ -+ defined(WL_OUTPUT_DESCRIPTION_SINCE_VERSION) -+ , display_handle_name, display_handle_desc -+#endif -+}; - - void Display::shm_format(void* /* data */, - struct wl_shm* /* wl_shm */, -diff --git a/shell/wayland/display.h b/shell/wayland/display.h -index cc3f4be..a0756f0 100644 ---- a/shell/wayland/display.h -+++ b/shell/wayland/display.h -@@ -329,6 +329,8 @@ class Display { - int32_t scale; - MAYBE_UNUSED bool done; - int transform; -+ std::string name; -+ std::string desc; - } output_info_t; - - struct pointer_event { -@@ -520,6 +522,32 @@ class Display { - */ - static void display_handle_done(void* data, struct wl_output* wl_output); - -+ /** -+ * @brief Set the display output name -+ * @param[in,out] data Data of type output_info_t* -+ * @param[in] wl_output No use -+ * @param[in] output_name Display name -+ * @return void -+ * @relation -+ * wayland - since @v4 of wl_output -+ */ -+ static void display_handle_name(void* data, -+ struct wl_output* wl_output, -+ const char* output_name); -+ -+ /** -+ * @brief Set the display description -+ * @param[in,out] data Data of type output_info_t* -+ * @param[in] wl_output No use -+ * @param[in] desc_name Display description name -+ * @return void -+ * @relation -+ * wayland - since @v4 of wl_output -+ */ -+ static void display_handle_desc(void* data, -+ struct wl_output* wl_output, -+ const char* desc_name); -+ - static const struct wl_shm_listener shm_listener; - - /** --- -2.35.1 - diff --git a/meta-agl-flutter/recipes-graphics/toyota/files/0002-activation_area-Allow-x-and-y-values-be-zero-for-the.patch b/meta-agl-flutter/recipes-graphics/toyota/files/0002-activation_area-Allow-x-and-y-values-be-zero-for-the.patch deleted file mode 100644 index f0cf9ddb3..000000000 --- a/meta-agl-flutter/recipes-graphics/toyota/files/0002-activation_area-Allow-x-and-y-values-be-zero-for-the.patch +++ /dev/null @@ -1,28 +0,0 @@ -From 641ca7ca26c1bfc11e7d0c0f30b731f53467bf1f Mon Sep 17 00:00:00 2001 -From: Marius Vlad <marius.vlad@collabora.com> -Date: Thu, 28 Dec 2023 20:28:12 +0200 -Subject: [PATCH 2/2] activation_area: Allow x and y values be zero for the - activation area - -Bug-AGL: SPEC-5038 -Signed-off-by: Marius Vlad <marius.vlad@collabora.com> ---- - shell/wayland/window.cc | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - -diff --git a/shell/wayland/window.cc b/shell/wayland/window.cc -index f816e58..929b9af 100644 ---- a/shell/wayland/window.cc -+++ b/shell/wayland/window.cc -@@ -100,7 +100,7 @@ WaylandWindow::WaylandWindow(size_t index, - break; - case WINDOW_BG: - m_display->AglShellDoBackground(m_base_surface, 0); -- if (m_activation_area.x > 0 && m_activation_area.y > 0) -+ if (m_activation_area.x >= 0 && m_activation_area.y >= 0) - m_display->AglShellDoSetupActivationArea( - m_activation_area.x, m_activation_area.y, m_activation_area.width, - m_activation_area.height, 0); --- -2.35.1 - diff --git a/meta-agl-flutter/recipes-graphics/toyota/files/0002-display-Add-support-for-agl_shell-version-8.patch b/meta-agl-flutter/recipes-graphics/toyota/files/0002-display-Add-support-for-agl_shell-version-8.patch deleted file mode 100644 index 792811769..000000000 --- a/meta-agl-flutter/recipes-graphics/toyota/files/0002-display-Add-support-for-agl_shell-version-8.patch +++ /dev/null @@ -1,477 +0,0 @@ -From d44f07a0c2cc410414bfd7b338ee071c17422a0a Mon Sep 17 00:00:00 2001 -From: Marius Vlad <marius.vlad@collabora.com> -Date: Mon, 4 Dec 2023 18:17:00 +0200 -Subject: [PATCH 2/2] display: Add support for agl_shell version 8 - -Signed-off-by: Marius Vlad <marius.vlad@collabora.com> ---- - shell/wayland/display.cc | 156 ++++++++++++++++++++++-- - shell/wayland/display.h | 58 +++++++++ - third_party/agl/protocol/agl-shell.xml | 160 ++++++++++++++++++++++++- - 3 files changed, 366 insertions(+), 8 deletions(-) - -diff --git a/shell/wayland/display.cc b/shell/wayland/display.cc -index 3ee814a..aba050a 100644 ---- a/shell/wayland/display.cc -+++ b/shell/wayland/display.cc -@@ -220,7 +220,7 @@ void Display::registry_handle_global(void* data, - if (version >= 2) { - d->m_agl.shell = static_cast<struct agl_shell*>( - wl_registry_bind(registry, name, &agl_shell_interface, -- std::min(static_cast<uint32_t>(4), version))); -+ std::min(static_cast<uint32_t>(8), version))); - agl_shell_add_listener(d->m_agl.shell, &agl_shell_listener, data); - } else { - d->m_agl.shell = static_cast<struct agl_shell*>( -@@ -980,6 +980,148 @@ void Display::agl_shell_bound_fail(void* data, struct agl_shell* shell) { - d->m_agl.bound_ok = false; - } - -+void Display::addAppToStack(std::string app_id) { -+ if (app_id == "homescreen") -+ return; -+ -+ bool found_app = false; -+ for (auto& i : apps_stack) { -+ if (i == app_id) { -+ found_app = true; -+ break; -+ } -+ } -+ -+ if (!found_app) { -+ apps_stack.push_back(app_id); -+ } else { -+ // fixme -+ } -+} -+ -+int Display::find_output_by_name(std::string output_name) { -+ int index = 0; -+ for (auto& i : m_all_outputs) { -+ if (i->name == output_name) { -+ return index; -+ } -+ index++; -+ } -+ -+ return -1; -+} -+ -+void Display::activateApp(std::string app_id) { -+ int default_output_index = 0; -+ -+ FML_LOG(INFO) << "got app_id " << app_id; -+ -+ // search for a pending application which might have a different output -+ auto iter = pending_app_list.begin(); -+ bool found_pending_app = false; -+ while (iter != pending_app_list.end()) { -+ auto app_to_search = iter->first; -+ FML_LOG(INFO) << "searching for " << app_to_search; -+ -+ if (app_to_search == app_id) { -+ found_pending_app = true; -+ break; -+ } -+ -+ iter++; -+ } -+ -+ if (found_pending_app) { -+ auto output_name = iter->second; -+ default_output_index = find_output_by_name(output_name); -+ -+ FML_LOG(INFO) << "Found app_id " << app_id << " at all"; -+ -+ if (default_output_index < 0) { -+ // try with remoting-remote-X which is the streaming -+ std::string new_remote_output = "remoting-" + output_name; -+ -+ default_output_index = find_output_by_name(new_remote_output); -+ if (default_output_index < 0) { -+ FML_LOG(INFO) << "Not activating app_id " << app_id << " at all"; -+ return; -+ } -+ } -+ -+ pending_app_list.erase(iter); -+ } -+ -+ FML_LOG(INFO) << "Activating app_id " << app_id << " on output " -+ << default_output_index; -+ agl_shell_activate_app(m_agl.shell, app_id.c_str(), -+ m_all_outputs[default_output_index]->output); -+ wl_display_flush(m_display); -+} -+ -+void Display::deactivateApp(std::string app_id) { -+ for (auto& i : apps_stack) { -+ if (i == app_id) { -+ // remove it from apps_stack -+ apps_stack.remove(i); -+ if (!apps_stack.empty()) -+ activateApp(apps_stack.back()); -+ break; -+ } -+ } -+} -+ -+void Display::processAppStatusEvent(const char* app_id, -+ const std::string event_type) { -+ if (!m_agl.shell) -+ return; -+ -+ if (event_type == "started") { -+ activateApp(std::string(app_id)); -+ } else if (event_type == "terminated") { -+ deactivateApp(std::string(app_id)); -+ } else if (event_type == "deactivated") { -+ // not handled -+ } -+} -+ -+void Display::agl_shell_app_on_output(void* data, -+ struct agl_shell* agl_shell, -+ const char* app_id, -+ const char* output_name) { -+ auto* d = static_cast<Display*>(data); -+ -+ FML_LOG(INFO) << "Gove event app_on_out app_id " << app_id << " output name " -+ << output_name; -+ -+ // a couple of use-cases, if there is no app_id in the app_list then it -+ // means this is a request to map the application, from the start to a -+ // different output that the default one. We'd get an -+ // AGL_SHELL_APP_STATE_STARTED which will handle activation. -+ // -+ // if there's an app_id then it means we might have gotten an event to -+ // move the application to another output; so we'd need to process it -+ // by explicitly calling processAppStatusEvent() which would ultimately -+ // activate the application on other output. We'd have to pick-up the -+ // last activated window and activate the default output. -+ // -+ // finally if the outputs are identical probably that's an user-error - -+ // but the compositor won't activate it again, so we don't handle that. -+ std::pair new_pending_app = -+ std::pair(std::string(app_id), std::string(output_name)); -+ d->pending_app_list.push_back(new_pending_app); -+ -+ auto iter = d->apps_stack.begin(); -+ while (iter != d->apps_stack.end()) { -+ if (*iter == std::string(app_id)) { -+ FML_LOG(INFO) << "Gove event to move " << app_id << " to another output " -+ << output_name; -+ d->processAppStatusEvent(app_id, std::string("started")); -+ break; -+ } -+ iter++; -+ } -+} -+ - void Display::agl_shell_app_state(void* data, - struct agl_shell* /* agl_shell */, - const char* app_id, -@@ -991,12 +1133,7 @@ void Display::agl_shell_app_state(void* data, - FML_DLOG(INFO) << "Got AGL_SHELL_APP_STATE_STARTED for app_id " << app_id; - - if (d->m_agl.shell) { -- // we always assume the first output advertised by the wl_output -- // interface -- unsigned int default_output_index = 0; -- -- agl_shell_activate_app(d->m_agl.shell, app_id, -- d->m_all_outputs[default_output_index]->output); -+ d->processAppStatusEvent(app_id, std::string("started")); - } - - break; -@@ -1007,6 +1144,10 @@ void Display::agl_shell_app_state(void* data, - case AGL_SHELL_APP_STATE_ACTIVATED: - FML_DLOG(INFO) << "Got AGL_SHELL_APP_STATE_ACTIVATED for app_id " - << app_id; -+ d->addAppToStack(std::string(app_id)); -+ break; -+ case AGL_SHELL_APP_STATE_DEACTIVATED: -+ d->processAppStatusEvent(app_id, std::string("deactivated")); - break; - default: - break; -@@ -1017,6 +1158,7 @@ const struct agl_shell_listener Display::agl_shell_listener = { - .bound_ok = agl_shell_bound_ok, - .bound_fail = agl_shell_bound_fail, - .app_state = agl_shell_app_state, -+ .app_on_output = agl_shell_app_on_output, - }; - - void Display::ivi_wm_surface_visibility(void* /* data */, -diff --git a/shell/wayland/display.h b/shell/wayland/display.h -index a0756f0..b919047 100644 ---- a/shell/wayland/display.h -+++ b/shell/wayland/display.h -@@ -18,6 +18,7 @@ - #pragma once - - #include <chrono> -+#include <list> - #include <memory> - #include <mutex> - #include <string> -@@ -271,6 +272,44 @@ class Display { - */ - std::pair<int32_t, int32_t> GetVideoModeSize(uint32_t index); - -+ /** -+ * @brief deactivate/hide the application pointed by app_id -+ * @param[in] app_id the app_id -+ * @relation -+ * agl_shell -+ */ -+ void deactivateApp(std::string app_id); -+ /** -+ * @brief activate/show the application pointed by app_id -+ * @param[in] app_id the app_id -+ * @relation -+ * agl_shell -+ */ -+ void activateApp(std::string app_id); -+ /** -+ * @brief Add app_id to a list of list applications -+ * @param[in] app_id the app_id -+ * @relation -+ * agl_shell -+ */ -+ void addAppToStack(std::string app_id); -+ /** -+ * @brief Helper to retrieve the output using its output_name -+ * @param[in] output_name a std::string representing the output -+ * @retval an integer that can used to get the proper output -+ * @relation -+ * agl_sell -+ */ -+ int find_output_by_name(std::string output_name); -+ /** -+ * @brief helper to process the application status -+ * @param[in] app_id an array of char -+ * @param[in] event_type a std::string representing the type of event (started/stopped/terminated) -+ * @relation -+ * agl_shell -+ */ -+ void processAppStatusEvent(const char* app_id, const std::string event_type); -+ - private: - std::shared_ptr<Engine> m_flutter_engine; - -@@ -300,6 +339,9 @@ class Display { - uint32_t version = 0; - } m_agl; - -+ std::list<std::string> apps_stack; -+ std::list<std::pair<const std::string, const std::string>> pending_app_list; -+ - struct ivi_shell { - struct ivi_application* application = nullptr; - struct ivi_wm* ivi_wm = nullptr; -@@ -982,6 +1024,22 @@ class Display { - const char* app_id, - uint32_t state); - -+ /** -+ * @brief AGL app_app_on_output event -+ * @param[in,out] data Data of type Display -+ * @param[in] shell No use -+ * @param[in] app_id the application id for which this event was sent -+ * @param[in] state the state: CREATED/TERMINATED/ACTIVATED/DEACTIVATED -+ * @return void -+ * @relation -+ * wayland, agl-shell -+ * @note Do nothing -+ */ -+ static void agl_shell_app_on_output(void* data, -+ struct agl_shell* agl_shell, -+ const char* app_id, -+ const char* output_name); -+ - static const struct agl_shell_listener agl_shell_listener; - - /** -diff --git a/third_party/agl/protocol/agl-shell.xml b/third_party/agl/protocol/agl-shell.xml -index bf5ab02..e010a80 100644 ---- a/third_party/agl/protocol/agl-shell.xml -+++ b/third_party/agl/protocol/agl-shell.xml -@@ -22,7 +22,7 @@ - FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER - DEALINGS IN THE SOFTWARE. - </copyright> -- <interface name="agl_shell" version="4"> -+ <interface name="agl_shell" version="8"> - <description summary="user interface for Automotive Grade Linux platform"> - Starting with version 2 of the protocol, the client is required to wait - for the 'bound_ok' or 'bound_fail' events in order to proceed further. -@@ -200,5 +200,163 @@ - <arg name="width" type="int" summary="width of rectangle"/> - <arg name="height" type="int" summary="height of rectangle"/> - </request> -+ -+ <request name="deactivate_app" since="5"> -+ <description summary="de-activate/hide window identified by app_id"> -+ Ask the compositor to hide the toplevel window for window -+ management purposes. Depending on the window role, this request -+ will either display the previously active window (or the background -+ in case there's no previously active surface) or temporarily (or -+ until a 'activate_app' is called upon) hide the surface. -+ -+ All the surfaces are identifiable by using the app_id, and no actions -+ are taken in case the app_id is not/was not present. -+ -+ See xdg_toplevel.set_app_id from the xdg-shell protocol for a -+ description of app_id. -+ </description> -+ <arg name="app_id" type="string"/> -+ </request> -+ -+ <request name="set_app_float" since="6"> -+ <description summary="set the window identified by app_id as float"> -+ Makes the application identified by app_id as floating. If the -+ application's window is already mapped, in a maximized, normal state, -+ it would transition to the float state. -+ -+ For applications that want to modify their own state, this request -+ must be done before the initial surface commit in order to take effect. -+ -+ If the application is already in floating state, this request wouldn't -+ do anything. -+ -+ There's no persistence of this request, once the application terminated -+ you'll to issue this request again for that particular app_id. -+ -+ The x, and y values would be initial position of the window where the -+ window surface will be placed. -+ -+ See xdg_toplevel.set_app_id from the xdg-shell protocol for a -+ description of app_id. -+ </description> -+ <arg name="app_id" type="string"/> -+ <arg name="x" type="int" summary="x position"/> -+ <arg name="y" type="int" summary="y position"/> -+ </request> -+ -+ <request name="set_app_normal" since="6"> -+ <description summary="set the window identified by app_id as normally started"> -+ Returns the application identified by app_id as it was in the normal state. -+ This is useful to come back from other states to the maximized state, the -+ normal state applications are started. -+ </description> -+ <arg name="app_id" type="string"/> -+ </request> -+ -+ <request name="set_app_fullscreen" since="7"> -+ <description summary=""> -+ Makes the application identified by app_id as fullscreen. If the -+ application's window is already mapped, in a maximized, normal state, -+ it would transition to the fullscreen state. -+ -+ For applications that want to modify their own state, this request -+ must be done before the initial surface commit in order to take effect. -+ -+ If the application is already in fullscreen state, this request wouldn't -+ do anything. -+ -+ There's no persistence of this request, once the application terminated -+ you'll to issue this request again for that particular app_id. -+ -+ See xdg_toplevel.set_app_id from the xdg-shell protocol for a -+ description of app_id. -+ </description> -+ <arg name="app_id" type="string"/> -+ </request> -+ -+ <request name="set_app_output" since="8"> -+ <description summary="Assign an application to a particular output"> -+ This would allow the compositor to place an application on a particular -+ output, if that output is indeed available. This can happen before -+ application is started which would make the application start on that -+ particular output. If the application is already started it would -+ move the application to that output. -+ -+ There's no persistence of this request, once the application terminated -+ you'll need to issue this request again for that particular app_id. -+ -+ See xdg_toplevel.set_app_id from the xdg-shell protocol for a -+ description of app_id. -+ </description> -+ <arg name="app_id" type="string"/> -+ <arg name="output" type="object" interface="wl_output"/> -+ </request> -+ -+ <event name="app_on_output" since="8"> -+ <description summary="Event sent as a reponse to set_app_output"> -+ Clients can use this event to be notified when an application -+ wants to be displayed on a certain output. This event is sent in -+ response to the set_app_output request. -+ -+ See xdg_toplevel.set_app_id from the xdg-shell protocol for a -+ description of app_id. -+ </description> -+ <arg name="app_id" type="string"/> -+ <arg name="output_name" type="string"/> -+ </event> -+ </interface> -+ -+ <interface name="agl_shell_ext" version="1"> -+ <description summary="extended user interface for Automotive Grade Linux platform"> -+ This interface allows another client bind to the agl_shell interface, -+ while there's another shell client already present. -+ -+ The client should first bind to this interface and then inform the -+ compositor with the 'doas_shell_client' request and it wants to bind to -+ the agl_shell interface. The client is still expected, if using a new -+ version of the agl_shell interface, to wait for the 'bound_ok' and -+ 'bound_fail' events before issueing any other requests/events. -+ -+ Note that this interface has its limitations, and the compositor would -+ still refuse the act for 'set_panel' or 'set_background' requests -+ of the agl_shell interface if there's already a client that used them. -+ -+ Any other requests or events should be delievered and handled as it would -+ a client bound to the agl_shell interface. -+ </description> -+ -+ <enum name="doas_shell_client_status"> -+ <entry name="success" value="0"/> -+ <entry name="failed" value="1"/> -+ </enum> -+ -+ <request name="destroy" type="destructor"> -+ <description summary="destroys the factory object"> -+ Call the destructor once you're ready with agl_shell_ext interface. -+ This would reset the state and would make any requests made -+ on the agl_shell interface be terminated. The client would need -+ to bind again the agl_shell_ext and issue a 'doas_shell_client' -+ request. -+ </description> -+ </request> -+ -+ <request name="doas_shell_client"> -+ <description summary="Informs the compositor it wants to bind to the -+ agl_shell interface"> -+ Prior to binding to agl_shell interface, this request would inform -+ the compositor that it wants to gain access the agl_shell interface. -+ The client is expected to wait for 'doas_shell_client_done' event and -+ check for a successful status before going further with binding to -+ the agl_shell interface. -+ </description> -+ </request> -+ -+ <event name="doas_done"> -+ <description summary="event sent as a reply to doas_shell_client"> -+ The client should check the status event to verify that the -+ compositor was able to handle the request. -+ </description> -+ <arg name="status" type="uint" enum="doas_shell_client_status"/> -+ </event> - </interface> - </protocol> --- -2.35.1 - diff --git a/meta-agl-flutter/recipes-graphics/toyota/files/0003-activation_area-Remove-the-implicit-width-height-swa.patch b/meta-agl-flutter/recipes-graphics/toyota/files/0003-activation_area-Remove-the-implicit-width-height-swa.patch deleted file mode 100644 index f59add5a8..000000000 --- a/meta-agl-flutter/recipes-graphics/toyota/files/0003-activation_area-Remove-the-implicit-width-height-swa.patch +++ /dev/null @@ -1,33 +0,0 @@ -From b247f63441f75c47cebd6edd00ecc0c5f94728b6 Mon Sep 17 00:00:00 2001 -From: Marius Vlad <marius.vlad@collabora.com> -Date: Tue, 2 Jan 2024 15:00:13 +0200 -Subject: [PATCH 3/3] activation_area: Remove the implicit width/height swap - -This does more harm than being useful so remove it. - -Bug-AGL: SPEC-5038 -Signed-off-by: Marius Vlad <marius.vlad@collabora.com> ---- - shell/wayland/display.cc | 6 ------ - 1 file changed, 6 deletions(-) - -diff --git a/shell/wayland/display.cc b/shell/wayland/display.cc -index 0b97954..9452dbf 100644 ---- a/shell/wayland/display.cc -+++ b/shell/wayland/display.cc -@@ -863,12 +863,6 @@ void Display::AglShellDoSetupActivationArea(uint32_t x, - if (!m_agl.shell) - return; - -- if (m_all_outputs[index]->transform == WL_OUTPUT_TRANSFORM_90) { -- uint32_t tmp_width = width; -- width = height; -- height = tmp_width; -- } -- - SPDLOG_DEBUG("Using custom rectangle [{}x{}+{}x{}] for activation", width, - height, x, y); - --- -2.35.1 - diff --git a/meta-agl-flutter/recipes-graphics/toyota/flutter-auto_aglflutter.inc b/meta-agl-flutter/recipes-graphics/toyota/flutter-auto_aglflutter.inc index c75cdc007..7ba85a34d 100644 --- a/meta-agl-flutter/recipes-graphics/toyota/flutter-auto_aglflutter.inc +++ b/meta-agl-flutter/recipes-graphics/toyota/flutter-auto_aglflutter.inc @@ -1,9 +1,3 @@ FILESEXTRAPATHS:prepend := "${THISDIR}/files:" SRC_URI:append:use-nxp-bsp = " file://0001-Disable-on_frame_base_surface-wl_surface_commit.patch" -SRC_URI:append: = " file://0001-display-Add-support-for-wl_output-version-4.patch \ - file://0002-display-Add-support-for-agl_shell-version-8.patch \ - file://0001-activation_area-Add-missing-width-and-height.patch \ - file://0002-activation_area-Allow-x-and-y-values-be-zero-for-the.patch \ - file://0003-activation_area-Remove-the-implicit-width-height-swa.patch \ - " diff --git a/meta-agl-flutter/tools/configs/_repos.json b/meta-agl-flutter/tools/configs/_repos.json index 7002d4161..5bc0562e2 100644 --- a/meta-agl-flutter/tools/configs/_repos.json +++ b/meta-agl-flutter/tools/configs/_repos.json @@ -1,8 +1,8 @@ [ { "uri": "https://github.com/toyota-connected/ivi-homescreen.git", - "rev": "ee49745485a88f23c32b8e3a62da40d4102c4623", - "branch": "agl" + "rev": "ea89b1de0a40c0d447ded305645556c978cfa696", + "branch": "v2.0" }, { "uri": "https://chromium.googlesource.com/chromium/tools/depot_tools.git", diff --git a/meta-agl-flutter/tools/configs/desktop-auto.json b/meta-agl-flutter/tools/configs/desktop-auto.json index cb3b9e688..943bca085 100644 --- a/meta-agl-flutter/tools/configs/desktop-auto.json +++ b/meta-agl-flutter/tools/configs/desktop-auto.json @@ -13,15 +13,15 @@ "env": { "HOMESCREEN_SRC_DIR": "${FLUTTER_WORKSPACE}/app/ivi-homescreen", "HOMESCREEN_BUILD_DIR": "${FLUTTER_WORKSPACE}/app/ivi-homescreen/build", - "HOMESCREEN_CMAKE_ARGS": "-DCMAKE_BUILD_TYPE=Debug -DENABLE_DLT=OFF", - "HOMESCREEN_EXE": "${HOMESCREEN_BUILD_DIR}/shell/homescreen --j=${PLATFORM_ID_DIR_RELATIVE}/default_config.json --b=${PLATFORM_ID_DIR_RELATIVE}", + "HOMESCREEN_CMAKE_ARGS": "-DCMAKE_BUILD_TYPE=Debug -DEXE_OUTPUT_NAME=flutter-auto -DSANITIZE_ADDRESS=OFF -DENABLE_DLT=OFF -DBUILD_UNIT_TESTS=OFF -DBUILD_DOCS=OFF -DBUILD_CRASH_HANDLER=OFF -DLLVM_ROOT=/usr -DBUILD_PLUGIN_FILAMENT_VIEW=OFF -DENABLE_IVI_SHELL_CLIENT=OFF -DENABLE_XDG_CLIENT=ON -DENABLE_AGL_CLIENT=ON -DENABLE_LTO=ON -DDEBUG_PLATFORM_MESSAGES=OFF -DBUILD_BACKEND_WAYLAND_EGL=ON -DBUILD_BACKEND_WAYLAND_VULKAN=OFF -DBUILD_EGL_TRANSPARENCY=ON -DBUILD_EGL_ENABLE_3D=ON -DBUILD_EGL_ENABLE_MULTISAMPLE=ON -DBUILD_PLUGIN_PDF=OFF -DBUILD_PLUGIN_FILAMENT_VIEW=OFF -DBUILD_PLUGIN_NAV_RENDER_VIEW=ON -DBUILD_PLUGIN_LAYER_PLAYGROUND_VIEW=ON -DBUILD_PLUGIN_AUDIOPLAYERS_LINUX=ON -DBUILD_PLUGIN_URL_LAUNCHER=ON -DBUILD_PLUGIN_FILE_SELECTOR=ON -DBUILD_PLUGIN_CAMERA=OFF -DBUILD_PLUGIN_GO_ROUTER=ON -DBUILD_PLUGIN_SECURE_STORAGE=ON", + "HOMESCREEN_EXE": "SPDLOG_LEVEL=debug ${HOMESCREEN_BUILD_DIR}/shell/flutter-auto --j=${PLATFORM_ID_DIR_RELATIVE}/default_config.json --b=${PLATFORM_ID_DIR_RELATIVE}", "PING_CMD": "loginctl show-session $(awk '/tty/ {print $1}' <(loginctl)) -p Type", "CREATE_BUNDLE_FOLDER": "mkdir -p ${PLATFORM_ID_DIR_RELATIVE}/data", "COPY_PLATFORM_ID_TO_BUNDLE_FOLDER": "cp -r ${PLATFORM_ID_DIR}/default_config.json ${PLATFORM_ID_DIR_RELATIVE}/", "COPY_BUNDLE_FOLDER_TO_PLATOFRM_ID": "cp -r ${BUNDLE_FOLDER}/* ${PLATFORM_ID_DIR_RELATIVE}/", "CREATE_ASSET_SYMLINK": "ln -rsf build/flutter_assets ${PLATFORM_ID_DIR_RELATIVE}/data/flutter_assets", "STAGE_BUNDLE_FOLDER": "${CREATE_BUNDLE_FOLDER} && ${COPY_PLATFORM_ID_TO_BUNDLE_FOLDER} && ${COPY_BUNDLE_FOLDER_TO_PLATOFRM_ID} && ${CREATE_ASSET_SYMLINK}" - }, + }, "runtime": { "config": { "view": [ @@ -41,9 +41,10 @@ "sudo add-apt-repository -y ppa:kisak/kisak-mesa", "sudo apt update -y", "sudo apt install lsb-release wget software-properties-common gnupg apt-file", - "sudo apt-file update", + "sudo apt update", "sudo apt install -y libwayland-dev wayland-protocols mesa-common-dev libegl1-mesa-dev libgles2-mesa-dev mesa-utils libxkbcommon-dev ninja-build", - "sudo apt install -y build-essential libvulkan-dev vulkan-tools vulkan-validationlayers vulkan-utils libtool", + "sudo apt install -y build-essential libvulkan-dev vulkan-tools vulkan-validationlayers vulkan-utils libtool libsecret-1-dev libsdl2-dev libjpeg-dev zenity", + "sudo apt install -y libllvm-ocaml-dev llvm llvm-dev llvm-runtime clang clang-tools libclang-dev libclang1 clang-format python3-clang clangd clang-tidy lldb lld libc++-dev libc++abi-dev libunwind-dev libc++1 libc++abi1", "sudo apt autoremove -y", "sudo apt clean -y", "cmake --version", @@ -54,7 +55,8 @@ "cmds": [ "sudo dnf -y update", "sudo dnf -y upgrade", - "sudo dnf -y install wayland-devel wayland-protocols-devel libxkbcommon-devel cmake ninja-build mesa-dri-drivers mesa-filesystem mesa-libEGL-devel mesa-libGL-devel mesa-libGLU-devel mesa-libgbm-devel mesa-libglapi mesa-libxatracker mesa-vulkan-drivers vulkan-tools libunwind-devel gstreamer1-devel gstreamer1-plugins-base-devel gstreamer1-plugins-bad-free-devel gstreamer1-plugins-bad-free-extras gstreamer1-plugins-base-tools gstreamer1-plugins-good gstreamer1-plugins-good-extras gstreamer1-plugins-ugly-free libtool", + "sudo dnf -y install wayland-devel wayland-protocols-devel libxkbcommon-devel cmake ninja-build mesa-dri-drivers mesa-filesystem mesa-libEGL-devel mesa-libGL-devel mesa-libGLU-devel mesa-libgbm-devel mesa-libglapi mesa-libxatracker mesa-vulkan-drivers vulkan-tools libunwind-devel gstreamer1-devel gstreamer1-plugins-base-devel gstreamer1-plugins-bad-free-devel gstreamer1-plugins-bad-free-extras gstreamer1-plugins-base-tools gstreamer1-plugins-good gstreamer1-plugins-good-extras gstreamer1-plugins-ugly-free", + "sudo dnf -y install libcxx-devel libcxx-static libcxxabi-devel libcxxabi-static ninja-build lld libtool SDL2-devel libsecret-devel zenity", "sudo dnf -y autoremove", "cmake --version", "dnf provides cmake" @@ -64,10 +66,14 @@ }, "post_cmds": [ { + "env": { + "CC": "/usr/bin/clang", + "CXX": "/usr/bin/clang++" + }, "cwd": "${HOMESCREEN_BUILD_DIR}", "cmds": [ - "cmake ${HOMESCREEN_SRC_DIR} ${HOMESCREEN_CMAKE_ARGS}", - "make homescreen -j" + "cmake -GNinja ${HOMESCREEN_SRC_DIR} ${HOMESCREEN_CMAKE_ARGS}", + "ninja homescreen" ] } ] @@ -75,7 +81,7 @@ "overwrite-existing": true, "custom-device": { "id": "desktop-auto", - "label": "Toyota Homescreen for AGL", + "label": "Toyota AGL flutter-auto", "sdkNameAndVersion": "flutter-auto x86_64", "platform": "linux-x64", "enabled": true, |