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

chore: cherry-pick e6e23ba00379 from chromium #38323

Merged
merged 2 commits into from May 16, 2023

Conversation

MarshallOfSound
Copy link
Member

Use ExternalBeginFrameSourceMac on macOS

Change ExternalBeginFrameSourceMac from being a
SyntheticBeginFrameSource to being an ExternalBeginFrameSource.

Move all of the code that is responsible for updating the VSync
parameters every 10 seconds from NativeWidgetMacNSWindowHost to
ExternalBeginFrameSourceMac.

Wire up ExternalBeginFrameSourceMac::SetVSyncDisplayID to create
the DisplayLinkMac (that was previously created in
NativeWidgetMacNSWindowHost). Set the VSyncCallbackMac callback in
ExternalBeginFrameSourceMac to update the timer based VSync
parameters the same way that was done in SyntheticBeginFrameSource.

Make RootCompositorFrameSinkImpl create a ExternalBeginFrameSourceMac
instead of creating a DelayBasedBeginFrameSource.

Bug: 1404797
Change-Id: I288497d94cc66356586e8da34852d53d05cf42f3

Notes: Backported fix for 1404797.

@MarshallOfSound MarshallOfSound requested a review from a team as a code owner May 16, 2023 17:35
@MarshallOfSound MarshallOfSound added semver/patch backwards-compatible bug fixes backport-check-skip Skip trop's backport validity checking main labels May 16, 2023
@electron-cation electron-cation bot added new-pr 🌱 PR opened in the last 24 hours and removed new-pr 🌱 PR opened in the last 24 hours labels May 16, 2023
@MarshallOfSound MarshallOfSound added the target/25-x-y PR should also be added to the "25-x-y" branch. label May 16, 2023
@MarshallOfSound
Copy link
Member Author

This fixes an issue on macOS where all compositors would be limited to 60fps regardless of monitor refresh rate

@VerteDinde VerteDinde merged commit 2a16c73 into main May 16, 2023
15 checks passed
@VerteDinde VerteDinde deleted the cherry-pick/main/chromium/e6e23ba00379 branch May 16, 2023 22:21
@release-clerk
Copy link

release-clerk bot commented May 16, 2023

Release Notes Persisted

Backported fix for 1404797.

@trop
Copy link
Contributor

trop bot commented May 16, 2023

I have automatically backported this PR to "25-x-y", please check out #38333

@trop trop bot added in-flight/25-x-y merged/25-x-y PR was merged to the "25-x-y" branch. and removed target/25-x-y PR should also be added to the "25-x-y" branch. in-flight/25-x-y labels May 16, 2023
ckerr pushed a commit that referenced this pull request May 17, 2023
* chore: cherry-pick e6e23ba00379 from chromium

* make it work
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-check-skip Skip trop's backport validity checking main merged/25-x-y PR was merged to the "25-x-y" branch. semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants