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

Introduce a custom wayland protocol for alien clients #214

Merged
merged 2 commits into from Nov 27, 2014

Conversation

giucam
Copy link
Contributor

@giucam giucam commented Nov 26, 2014

This PR introduces a wayland protocol, alien-manager.xml, used to handle non native, alien clients. Since they don't connect to lipstick directly but to a proxy process instead, the wl_shell protocol doesn't work well enough and introduces bugs.

@faenil
Copy link
Member

faenil commented Nov 26, 2014

would you mind elaborating more about the issues that this PR fixes?

@giucam
Copy link
Contributor Author

giucam commented Nov 26, 2014

Well, by itself it doesn't fix anything of course. But clients using it will be able to deal better with low memory situations with the oom_score event, or to have the covers work better thanks to the different states of alien_surface, without needing to use separate surfaces.

@jpetrell
Copy link

LGTM

@giucam giucam force-pushed the alien branch 5 times, most recently from d788c8b to 6050898 Compare November 27, 2014 09:17
@giucam giucam merged commit 07dc9cd into nemomobile:master Nov 27, 2014
giucam added a commit that referenced this pull request Nov 27, 2014
[lipstick] Introduce a custom wayland protocol for alien clients
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants