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

--permission-lifetime-test-seconds=20 doesn't appear to work on Android nightly builds #15523

Closed
stephendonner opened this issue Apr 27, 2021 · 2 comments
Labels
closed/duplicate Issue has already been reported

Comments

@stephendonner
Copy link

stephendonner commented Apr 27, 2021

Description

--permission-lifetime-test-seconds=20 doesn't appear to work on Android nightly builds

Steps to Reproduce

  1. launch nightly on Android
  2. enable Permission Lifetimes via brave://flags
  3. open Settings > Developer Options > QA Preferences
  4. enter the code (ping me on Slack if you don't have it)
  5. input --permission-lifetime-test-seconds=20 into the Enter Command Line string textfield
  6. tap OK
  7. tap on RELAUNCH NOW
  8. open https://permission.site
  9. tap on Camera + Microphone
  10. look at the radio-button options under Give permission

Actual result:

There's no 20 seconds option to choose from amongst the radio buttons for Give permission

example example example
Screenshot_20210427-145940 Screenshot_20210427-145954 Screenshot (Apr 27, 2021 3 10 53 PM)

Expected result:

There should be a 20 seconds option below forever, as there is on desktop

Screen Shot 2021-04-27 at 3 07 57 PM

Reproduces how often:

100%

Brave version (brave://version info)

Brave - Nightly 1.25.42, Chromium 90.0.4430.85 on Android 9

@goodov @deeppandya am I doing this right? Thanks in advance for looking! 👍

@goodov
Copy link
Member

goodov commented Apr 28, 2021

This cmdline stub mechanism ignores the first parameter. This is not specific to permission-lifetime-test-seconds, any parameter will be ignored. To test things you can use this: _ --permission-lifetime-test-seconds=10 as per: the first command line item should be a "_" (underscore) followed by the ones you actually need.

@goodov goodov added OS/Android Fixes related to Android browser functionality and removed OS/Desktop labels Apr 28, 2021
@goodov
Copy link
Member

goodov commented Apr 28, 2021

Created to fix this behavior. #15540

@goodov goodov closed this as completed Apr 28, 2021
@kjozwiak kjozwiak added closed/duplicate Issue has already been reported and removed OS/Android Fixes related to Android browser functionality labels Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/duplicate Issue has already been reported
Projects
None yet
Development

No branches or pull requests

3 participants