Skip to content

Add snippets for browsers for Android for Cars#882

Merged
jankleinert merged 3 commits into
mainfrom
jkleinert/browser
Apr 23, 2026
Merged

Add snippets for browsers for Android for Cars#882
jankleinert merged 3 commits into
mainfrom
jkleinert/browser

Conversation

@jankleinert
Copy link
Copy Markdown
Contributor

Migrate 2 snippets on the browsers page.

@snippet-bot
Copy link
Copy Markdown

snippet-bot Bot commented Apr 22, 2026

Here is the summary of changes.

You are about to add 2 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

Copy link
Copy Markdown
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 introduces utility functions to check if a device credential is set and to open the system security settings screen. The review feedback identifies a potential runtime exception in the openSecurityScreen function when using a non-Activity context, recommending the addition of the FLAG_ACTIVITY_NEW_TASK flag to the intent.

Comment thread cars/src/main/java/com/example/cars/parked/Browser.kt Outdated
Comment thread cars/src/main/java/com/example/cars/parked/Browser.kt Outdated
Comment thread cars/src/main/java/com/example/cars/parked/Browser.kt Outdated
@kkuan2011
Copy link
Copy Markdown
Contributor

looks good, thanks Jan! feel free to merge!

Copy link
Copy Markdown

@thomasweathers thomasweathers left a comment

Choose a reason for hiding this comment

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

LGTM! feel free to merge

@jankleinert jankleinert merged commit 45b656c into main Apr 23, 2026
5 checks passed
@jankleinert jankleinert deleted the jkleinert/browser branch April 23, 2026 17:58
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