Skip to content

perf: use GIO for Browser::IsDefaultProtocolClient() on Linux (#51316)#51627

Merged
samuelmaddock merged 2 commits into
42-x-yfrom
perf/faster-app.isDefaultProtocolClient-on-Linux--42-x-y
May 28, 2026
Merged

perf: use GIO for Browser::IsDefaultProtocolClient() on Linux (#51316)#51627
samuelmaddock merged 2 commits into
42-x-yfrom
perf/faster-app.isDefaultProtocolClient-on-Linux--42-x-y

Conversation

@ckerr

@ckerr ckerr commented May 14, 2026

Copy link
Copy Markdown
Member

Manually backport #51316 to 42-x-y. See that PR for details.

Notes: Improved performance of app.isDefaultProtocolClient() and app.setAsDefaultProtocolClient() on Linux.

@electron-cation electron-cation Bot added the new-pr 🌱 PR opened recently label May 14, 2026
@trop trop Bot added 42-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels May 14, 2026
@ckerr ckerr marked this pull request as draft May 14, 2026 21:21
@electron-cation electron-cation Bot removed the new-pr 🌱 PR opened recently label May 14, 2026
@ckerr ckerr force-pushed the perf/faster-app.isDefaultProtocolClient-on-Linux--42-x-y branch 2 times, most recently from f753473 to 2174d39 Compare May 20, 2026 02:11
@ckerr ckerr marked this pull request as ready for review May 20, 2026 14:34
* perf: use GIO for Browser::IsDefaultProtocolClient() on Linux

perf: use GIO for Browser::SetAsDefaultProtocolClient() on Linux

Similar to 7d6227a, this speeds up app.isDefaultProtocolClient()
by using the GIO library instead of spawning a shell command to
get the info.

* feat: log errors if g_app_info_set_as_default_for_type() fails
@samuelmaddock samuelmaddock force-pushed the perf/faster-app.isDefaultProtocolClient-on-Linux--42-x-y branch from 2174d39 to 332d286 Compare May 27, 2026 20:13
@samuelmaddock

Copy link
Copy Markdown
Member

macos test failing due to unrelated changes

Failure in test: "autoUpdater behavior with update server should hit the download endpoint when an update is available and update successfully when the zip is provided even after a different update was staged"

@samuelmaddock samuelmaddock enabled auto-merge (squash) May 28, 2026 15:01
@samuelmaddock samuelmaddock added fast-track 🚅 Indicates that this PR is intended to bypass the 24 hour rule. Needs approval from Releases and removed fast-track 🚅 Indicates that this PR is intended to bypass the 24 hour rule. Needs approval from Releases labels May 28, 2026
@samuelmaddock samuelmaddock merged commit be92890 into 42-x-y May 28, 2026
170 of 175 checks passed
@samuelmaddock samuelmaddock deleted the perf/faster-app.isDefaultProtocolClient-on-Linux--42-x-y branch May 28, 2026 15:59
@release-clerk

release-clerk Bot commented May 28, 2026

Copy link
Copy Markdown

Release Notes Persisted

Improved performance of app.isDefaultProtocolClient() and app.setAsDefaultProtocolClient() on Linux.

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

Labels

42-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