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

Add support for CEF128 #231

Merged
merged 1 commit into from
Aug 23, 2024
Merged

Add support for CEF128 #231

merged 1 commit into from
Aug 23, 2024

Conversation

HashidaTKS
Copy link
Contributor

@HashidaTKS HashidaTKS commented Aug 22, 2024

Which issue(s) this PR fixes:

#228

What this PR does / why we need it:

Add support for CEF128.

settings.chrome_runtime and settings.persist_user_preferences are removed since CEF128 because CEF always uses Chrome runtime, means chrome_runtime is always true, and persist_user_preferences is always true.

chromiumembedded/cef@a461a89#diff-447d40892c51b4ffc870cf1a28bab2ae2d411752216be4750a17193e6675b26bL244
chromiumembedded/cef@a461a89#diff-447d40892c51b4ffc870cf1a28bab2ae2d411752216be4750a17193e6675b26bL347
chromiumembedded/cef#3685

How to verify the fixed issue:

  • Confirm that build for CEF128 is passed.
    • This PR focuses to pass the build job, not confirming that the all Chronos functions work fine or not.

https://github.com/ThinBridge/Chronos/actions/runs/10505750334/job/29103948165

@HashidaTKS HashidaTKS requested a review from kenhys August 23, 2024 00:59
@HashidaTKS HashidaTKS marked this pull request as ready for review August 23, 2024 00:59
@HashidaTKS
Copy link
Contributor Author

@kenhys

Could you review this when you have time?

Copy link
Contributor

@kenhys kenhys left a comment

Choose a reason for hiding this comment

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

LGTM.

@kenhys kenhys merged commit 5ec2251 into master Aug 23, 2024
3 checks passed
@kenhys kenhys deleted the add-support-for-CEF128 branch August 23, 2024 02:17
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.

2 participants