Skip to content

fix: ensure that chromium binds to wayland interface versions it supports#32652

Merged
zcbenz merged 1 commit intoelectron:13-x-yfrom
Molytho:13-x-y
Jan 31, 2022
Merged

fix: ensure that chromium binds to wayland interface versions it supports#32652
zcbenz merged 1 commit intoelectron:13-x-yfrom
Molytho:13-x-y

Conversation

@Molytho
Copy link
Copy Markdown

@Molytho Molytho commented Jan 27, 2022

Chromium tries to bind to the newest version of some wayland interfaces. This causes issues when it doesn't actually support the version.
This patch ensures that only supported versions are bound to.

Fixes #32487
Backport of #32603

Notes: none.

@welcome
Copy link
Copy Markdown

welcome bot commented Jan 27, 2022

💖 Thanks for opening this pull request! 💖

We use semantic commit messages to streamline the release process. Before your pull request can be merged, you should update your pull request title to start with a semantic prefix.

Examples of commit messages with semantic prefixes:

  • fix: don't overwrite prevent_default if default wasn't prevented
  • feat: add app.isPackaged() method
  • docs: app.isDefaultProtocolClient is now available on Linux

Things that will help get your PR across the finish line:

  • Follow the JavaScript, C++, and Python coding style.
  • Run npm run lint locally to catch formatting errors earlier.
  • Document any user-facing changes you've made following the documentation styleguide.
  • Include tests when adding/changing behavior.
  • Include screenshots and animated GIFs whenever possible.

We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can.

@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Jan 27, 2022
@Molytho Molytho changed the title Ensure that chromium binds to wayland interface versions it supports fix: Ensure that chromium binds to wayland interface versions it supports Jan 28, 2022
@Molytho Molytho marked this pull request as ready for review January 28, 2022 20:01
@Molytho Molytho requested a review from a team as a code owner January 28, 2022 20:01
This patch is required to fix Ozone/Wayland on recent compositors (e.g. wlroots 15).
Chromium tries to bind to interface versions it does not support. This wasn't a problem
until wl_output version 4 got released because chromium supported every version up to this.

Fix #32487.
Upstream chromium issue: https://bugs.chromium.org/p/chromium/issues/detail?id=1279574
@trop trop bot added 13-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Jan 29, 2022
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jan 29, 2022
@trop trop bot added semver/patch backwards-compatible bug fixes and removed semver/patch backwards-compatible bug fixes labels Jan 31, 2022
@codebytere codebytere changed the title fix: Ensure that chromium binds to wayland interface versions it supports fix: ensure that chromium binds to wayland interface versions it supports Jan 31, 2022
@trop trop bot added semver/patch backwards-compatible bug fixes and removed semver/patch backwards-compatible bug fixes labels Jan 31, 2022
@zcbenz zcbenz merged commit ac71f57 into electron:13-x-y Jan 31, 2022
@welcome
Copy link
Copy Markdown

welcome bot commented Jan 31, 2022

Congrats on merging your first pull request! 🎉🎉🎉

@release-clerk
Copy link
Copy Markdown

release-clerk bot commented Jan 31, 2022

No Release Notes

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

Labels

13-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants