Skip to content

[tizen_window_manager] Add tizen-core-wl window backend - #1136

Closed
JSUYA wants to merge 4 commits into
flutter-tizen:mainfrom
JSUYA:tizen-core-wl-window-backend
Closed

[tizen_window_manager] Add tizen-core-wl window backend#1136
JSUYA wants to merge 4 commits into
flutter-tizen:mainfrom
JSUYA:tizen-core-wl-window-backend

Conversation

@JSUYA

@JSUYA JSUYA commented Sep 3, 2026

Copy link
Copy Markdown
Member

Introduce a WindowProxyInterface abstraction with two backends selected at runtime by WindowProxy:

  • TizenCoreWlWindowProxy (tizen-core-wl) on Tizen API version >= 11.0 when the library is available
  • EcoreWl2WindowProxy (ecore_wl2) otherwise, including as a fallback when tizen-core-wl cannot be loaded on 11.0+

Both backends implement WindowProxyInterface (GetGeometry/Activate/Lower) and resolve their symbols at runtime via dlopen/dlsym. TizenWindowManager constructs WindowProxy, which picks the backend by querying the platform version at startup.

@JSUYA
JSUYA marked this pull request as draft September 3, 2026 11:00
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-03T11:06:49.749910Z fa7652e PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Introduce a WindowProxyInterface abstraction with two backends selected at
runtime by WindowProxy:
  - TizenCoreWlWindowProxy (tizen-core-wl) on Tizen API version >= 11.0 when
    the library is available
  - EcoreWl2WindowProxy (ecore_wl2) otherwise, including as a fallback when
    tizen-core-wl cannot be loaded on 11.0+

Both backends implement WindowProxyInterface (GetGeometry/Activate/Lower) and
resolve their symbols at runtime via dlopen/dlsym. TizenWindowManager
constructs WindowProxy, which picks the backend by querying the platform
version at startup.
Log an error with get_error_message() when
tizen_core_wl_window_get_geometry/activate/lower returns a non-zero
error code, instead of silently ignoring failures.
Prevents uninitialized values from being returned to Dart when
GetGeometry fails (invalid handle or backend API error).
@JSUYA

JSUYA commented Sep 4, 2026

Copy link
Copy Markdown
Member Author

#1137

@JSUYA JSUYA closed this Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant