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

Redundancies in Command-Line Values #8263

Open
jonathansampson opened this issue Feb 15, 2020 · 3 comments
Open

Redundancies in Command-Line Values #8263

jonathansampson opened this issue Feb 15, 2020 · 3 comments
Labels
misc/command-line-args needs-more-info The report requires more detail before we can decide what to do with this issue. priority/P5 Not scheduled. Don't anticipate work on this any time soon.

Comments

@jonathansampson
Copy link
Contributor

Description

The Command-Line value in about:version often contains redundancies.

Steps to Reproduce

  1. Navigate to about:version

Actual result:

Redundancies in Command-Line value.

1.3.115 (current stable) contains the following redundancies:

--disable-features= AllowPopupsDuringPageUnload, AudioServiceOutOfProcess, AutofillServerCommunication, LookalikeUrlNavigationSuggestionsUI, NotificationTriggers, SmsReceiver, VideoPlaybackQuality, WebXR, WebXrGamepadModule
--disable-features= AllowPopupsDuringPageUnload, AudioServiceOutOfProcess, AutofillServerCommunication, LookalikeUrlNavigationSuggestionsUI, NotificationTriggers, SmsReceiver, VideoPlaybackQuality, WebXR, WebXrGamepadModule
--disable-webrtc-apm-in-audio-service
--disable-webrtc-apm-in-audio-service
--enable-audio-service-sandbox
--enable-audio-service-sandbox
--enable-features= PasswordImport, SimplifyHttpsIndicator, WebUIDarkMode
--enable-features= PasswordImport, SimplifyHttpsIndicator, WebUIDarkMode

1.4.86 (current beta) contains no redundancies.
1.5.91 (current developer) contains no redundancies.
1.6.32 (current nightly) contains the following redundancies:

--enable-features= BraveNTPBrandedWallpaperName, DnsOverHttps, PasswordImport, SimplifyHttpsIndicator, WebUIDarkMode
--enable-features=                               DnsOverHttps, PasswordImport, SimplifyHttpsIndicator, WebUIDarkMode

Expected result:

No unnecessary redundancies.

Reproduces how often:

Always.

Brave version (brave://version info)

Brave 1.3.115 Chromium: 80.0.3987.87 (Official Build) (64-bit)
Revision 449cb163497b70dbf98d389f54e38e85d4c59b43-refs/branch-heads/3987@{#801}

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? No
  • Can you reproduce this issue with the dev channel? No
  • Can you reproduce this issue with the nightly channel? Yes
@rebron
Copy link
Collaborator

rebron commented Apr 17, 2020

@jonathansampson
Can you try again with 1.7.92? Still an issue?

@rebron rebron added the needs-more-info The report requires more detail before we can decide what to do with this issue. label Apr 17, 2020
@jonathansampson
Copy link
Contributor Author

@rebron I still see redundancies:

"C:\Program Files (x86)\BraveSoftware\Brave-Browser\Application\brave.exe"
  --profile-directory=Default
  --enable-dom-distiller
  --disable-domain-reliability
  --no-pings
  --extension-content-verification=enforce_strict
  --extensions-install-verification=enforce
  --sync-url=https://no-thanks.invalid
> --enable-features=PasswordImport,WebUIDarkMode,AutoupgradeMixedContent,PassiveMixedContentWarning,MixedContentSiteSetting,SimplifyHttpsIndicator,DnsOverHttps
  --disable-features=AutofillServerCommunication,LookalikeUrlNavigationSuggestionsUI,NotificationTriggers,AllowPopupsDuringPageUnload,AudioServiceOutOfProcess,SmsReceiver,VideoPlaybackQuality
  --flag-switches-begin
> --enable-features=PasswordImport,WebUIDarkMode,AutoupgradeMixedContent,PassiveMixedContentWarning,MixedContentSiteSetting,SimplifyHttpsIndicator,DnsOverHttps,BraveSync
  --flag-switches-end
  --enable-audio-service-sandbox
  --disable-webrtc-apm-in-audio-service

@darkdh
Copy link
Member

darkdh commented Aug 12, 2020

It is tied to "relaunch" because it will inherit the args before relaunching
STR:

  1. Go to brave://flags/ and randomly change something and change back to trigger relaunch prompt

Screen Shot 2020-08-12 at 11 37 04

  1. Click Relaunch button
  2. After relaunch go to brave://version/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
misc/command-line-args needs-more-info The report requires more detail before we can decide what to do with this issue. priority/P5 Not scheduled. Don't anticipate work on this any time soon.
Projects
Status: Needs Info
Development

No branches or pull requests

4 participants