Skip to content

Conversation

@cobalt-github-releaser-bot
Copy link
Collaborator

Refer to the original PR: #9037

Move splash screen enablement check to command line.

This change configures the --disable-splash-screen flag based on app
metadata. This allows the Cobalt engine to skip creating the splash
screen WebContents object entirely when it is not needed, which
optimizes resource usage by avoiding unnecessary object creation.

Issue: 483157506

@cobalt-github-releaser-bot
Copy link
Collaborator Author

MERGE CONFLICT CAT

Caution

There were merge conflicts while cherry picking! Check out cherry-pick-26.android-9037 and fix the conflicts before proceeding. Check the log at https://github.com/youtube/cobalt/actions/runs/21887680273 for details.

@github-actions
Copy link

🤖 Gemini Suggested Commit Message


android: Integrate disable splash screen flag

Configure the --disable-splash-screen flag using app metadata.
This allows the Cobalt engine to entirely skip creating the splash
screen WebContents object when not needed, optimizing resource use.

Bug: 483157506

💡 Pro Tips for a Better Commit Message:

  1. Influence the Result: Want to change the output? You can write custom prompts or instructions directly in the Pull Request description. The model uses that text to generate the message.
  2. Re-run the Generator: Post a comment with: /generate-commit-message

Move splash screen enablement check to command line.

This change configures the --disable-splash-screen flag based on app
metadata. This allows the Cobalt engine to skip creating the splash
screen WebContents object entirely when it is not needed, which
optimizes resource usage by avoiding unnecessary object creation.

Issue: 483157506
(cherry picked from commit 3a82057)
@borongc borongc force-pushed the cherry-pick-26.android-9037 branch from 4b41501 to 70f920c Compare February 11, 2026 00:30
@borongc borongc marked this pull request as ready for review February 11, 2026 00:31
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request cherry-picks a change to move the splash screen enablement check to a command-line flag. The changes look correct and successfully apply the intended logic. I've added one suggestion to improve the efficiency and robustness of the newly added appendArgsFromMetaData method by avoiding unnecessary object allocations and preventing duplicate command-line flags.

@borongc borongc requested a review from johnxwork February 11, 2026 17:11
@johnxwork johnxwork merged commit 6afc6a2 into 26.android Feb 11, 2026
210 of 258 checks passed
@johnxwork johnxwork deleted the cherry-pick-26.android-9037 branch February 11, 2026 22:00
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.

3 participants