Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wayland-egl backend (previously named STM) #79

Merged
merged 5 commits into from
Aug 1, 2016

Conversation

blino
Copy link

@blino blino commented Aug 1, 2016

This backend creates wayland-egl window from the renderer process.
The view part only acks buffer to send frame complete events.

This is based on the STM implementation in WPE-old, and reuses the
Wayland::Display class initially implemented for BCMNexusWL.

Input support is not implemented yet.

This closes #65.

blino added 5 commits August 1, 2016 11:30
The STM is just a generic backend relying on wayland-egl, with no code
specific to STM chipsets.
This backend creates wayland-egl window from the renderer process.
The view part only acks buffer to send frame complete events.

This is based on the STM implementation in WPE-old, and reuses the
Wayland::Display class initially implemented for BCMNexusWL.

Input support is not implemented yet.

This closes WebPlatformForEmbedded#65.
@blino blino mentioned this pull request Aug 1, 2016
@zdobersek
Copy link

The code is pretty straight-forward, no complaints there. I don't know about the name though -- maybe it should be more clear that this is a special case, where the UIProcess doesn't act as a normal client of the compositor but it's instead the WebProcess that connects to it.

I don't know what STM stood for though, and I'm not terribly against WAYLAND_EGL. If you think there's a more descriptive name, feel free to switch to that.

@zdobersek zdobersek merged commit a1138d5 into WebPlatformForEmbedded:master Aug 1, 2016
@blino
Copy link
Author

blino commented Aug 1, 2016

STM probably stood for STMicroelectronics, they previously had a wide range of set-top-box products.
I don't have a better name in mind yet.

Thanks for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants