Architecture: adding windows on ARM support

thirumalai-qcom commented 1 month ago

Description

This pull request updates the OBS-browser module to the latest CEF (Chromium Embedded Framework) library. This update necessitates changes to existing function declarations and signatures to align with the new library's specifications. Additionally, ARM architecture support has been introduced, with modifications guarded by ARM-specific macros to ensure compatibility without affecting the existing codebase.

Motivation and Context

The primary motivation for this change is to keep the OBS-browser module up-to-date with the latest CEF library with the new features. Additionally, expanding compatibility to include ARM-based systems allows a broader range of users to benefit from the OBS-browser functionalities.

How Has This Been Tested?

The changes have been tested by building OBS Studio on an ARM machine (X elite machine). All builds were successful, Basic functionality tests confirmed that the OBS Studio application with the modified obs-browser is functional.

Types of changes

Library Update:
- Updated to the latest CEF library, resulting in changes to existing function declarations and structures.
- High-DPI support is now enabled by default in Chromium, resulting in the removal of the "CefEnableHighDPISupport" method.

Checklist:

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

RytoEX commented 1 month ago

This will need to be rebased now that #434 has been merged.

thirumalai-qcom commented 1 month ago

This will need to be rebased now that #434 has been merged.

Hi Ryan,

I have reviewed the mainline, and it seems that the changes in this PR have already been merged through PR #434. Could you please let me know if there are any specific reasons or additional changes you would like to see in this PR that necessitate a rebase?

RytoEX commented 1 month ago

If all changes in this PR are covered or superseded by #434, then this PR can simply be closed.

thirumalai-qcom commented 1 month ago

If all changes in this PR are covered or superseded by #434, then this PR can simply be closed.

I have confirmed that all changes in this PR have been covered by the changes merged from PR #434. Therefore, i will proceed to close this PR as it is no longer necessary.

obsproject / obs-browser