Skip to content

CI: Update CEF from 103/5060 to 127/6533#11213

Merged
RytoEX merged 1 commit into
obsproject:masterfrom
RytoEX:update-cef-to-127
Aug 31, 2024
Merged

CI: Update CEF from 103/5060 to 127/6533#11213
RytoEX merged 1 commit into
obsproject:masterfrom
RytoEX:update-cef-to-127

Conversation

@RytoEX

@RytoEX RytoEX commented Aug 28, 2024

Copy link
Copy Markdown
Member

Description

Update CEF from 103/5060 to 127/6533.

Motivation and Context

Many thanks to @reitowo , @WizardCM , @tytan652 , @pkviet , @PatTheMav , @kkartaltepe , @Lain-B , and everyone else who pitched in to answer questions, test, help, and build the packages so that we can update CEF.

How Has This Been Tested?

Built on Windows 11 and ran through some tests with browser sources (FishGL, HTML5Test, CSS3Test, etc.) and browser panels (Twitch Authentication).

More testing is welcome.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Tweak (non-breaking change to improve existing functionality)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@RytoEX RytoEX added kind/bug Categorizes issue or PR as related to a bug. kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. Seeking Testers Build artifacts on CI labels Aug 28, 2024
@RytoEX RytoEX added this to the OBS Studio 31 milestone Aug 28, 2024
@RytoEX RytoEX self-assigned this Aug 28, 2024
@RytoEX

RytoEX commented Aug 28, 2024

Copy link
Copy Markdown
Member Author

There will probably be another obs-browser update before this lands. I just wanted to get the PR up so that people could hopefully more easily test.

@RytoEX RytoEX force-pushed the update-cef-to-127 branch from 01bbab7 to ad14e0a Compare August 29, 2024 03:00

@WizardCM WizardCM left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on Windows, everything works as expected.

@RytoEX

RytoEX commented Aug 30, 2024

Copy link
Copy Markdown
Member Author

This should now be unblocked by 16011f7.

@RytoEX RytoEX merged commit 3559fda into obsproject:master Aug 31, 2024
@RytoEX RytoEX deleted the update-cef-to-127 branch August 31, 2024 02:38
@nproctorGV

nproctorGV commented Nov 21, 2024

Copy link
Copy Markdown

Have you noticed the VRAM leak on Windows?

@pkviet

pkviet commented Nov 21, 2024

Copy link
Copy Markdown
Member

Have you noticed the VRAM leak on Windows?

In early tests, iirc, there used to be a leak but it's been solved.
Are you saying it's still happening in beta ? If so, please file a bug with relevant details. And try to use clean default settings without any third party plugins.

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

Labels

kind/bug Categorizes issue or PR as related to a bug. kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. Seeking Testers Build artifacts on CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants