Skip to content

Conversation

@julenuri
Copy link
Collaborator

No description provided.

@julenuri julenuri requested a review from whindsaks July 10, 2025 15:31
@julenuri julenuri self-assigned this Jul 10, 2025
URLDownload = https://archive.org/download/48.0.2564.109-chrome-installer/chrome48_x64.zip
SHA1 = 18a47a2285f05665e3ad19446186cfbcc30e5ba3
SizeBytes = 80979795
Installer = Generate
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this actually work? Last time I tried Chrome crashed on startup (although that was x64). Do we need a feature to add parameters to the shortcut to disable the browser sandbox?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Exactly the same parameter that needs the x86 version of Chromium and Chrome.

Copy link
Contributor

@whindsaks whindsaks Jul 10, 2025

Choose a reason for hiding this comment

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

I don't know what that parameter is. But I can add the shortcut parameter feature to Rapps.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I mean, --no-sandbox it's compulsory in every Chromium based engine.

Copy link
Contributor

Choose a reason for hiding this comment

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

I just use Firefox in ROS.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Bad thing is Firefox in MSVC x64 has a biiiig bad bug: https://jira.reactos.org/browse/CORE-18341
Weirdly not in GCC x64. So... I have to investigate it again, somewhen.

julenuri and others added 2 commits July 10, 2025 20:07
Co-authored-by: Whindmar Saksit <[email protected]>
Co-authored-by: Whindmar Saksit <[email protected]>
@julenuri julenuri merged commit 25b2aca into reactos:master Jul 10, 2025
1 check passed
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