-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Make Brave Rewards extension non-persistent #6462
Comments
Right now this is difficult to achieve because the extension:
Instead, #3436 is more achievable and achieves the result of non-rewards users not having rewards-related memory use. |
Verification in progress with
Verified test plan from brave/brave-core#6978. Each of the below cases/sub cases was completed using a fresh profile. Website - laurenwags.github.io
YouTube - youtube.com/user/jjdsampson
YouTube - https://www.youtube.com/channel/UCD7siefto4I33L4U61IU5LQ
YouTube - https://www.youtube.com/watch?v=RFUt3wDGt-w
Twitter - twitter.com/emerick
GitHub - https://github.com/brave/brave-browser/commits
GitHub - https://github.com/LaurenWags
GitHub - https://github.com/brave/brave-browser/ pull/12811
GitHub - https://github.com/brave/brave-browser/ issues/12771
Reddit - https://www.reddit.com/user/alex_the_brave
Reddit - https://old.reddit.com/r/brave_browser/comments/g79s8n/is_there_a_way_to_remove_the_tip_button/
Vimeo - https://vimeo.com/bravelaurenwags
Verification passed on
Verified test plan from brave/brave-core#6978. Each of the below cases/sub cases was completed using a fresh profile. Website - DDG
YouTube - youtube.com/user/jjdsampson
Confirmed clicking on the BAT logo in URL bar to open the panel started the Rewards extension YouTube - https://www.youtube.com/channel/UCD7siefto4I33L4U61IU5LQ
Confirmed clicking on the BAT logo in URL bar to open the panel started the Rewards extension YouTube - https://www.youtube.com/watch?v=RFUt3wDGt-w
Confirmed clicking on the BAT logo in URL bar to open the panel started the Rewards extension Twitter - twitter.com/emerick
GitHub - https://github.com/brave/brave-browser/commits
GitHub - https://github.com/LaurenWags
GitHub - https://github.com/brave/brave-browser/ pull/12811
GitHub - https://github.com/brave/brave-browser/ issues/12771
Reddit - https://www.reddit.com/user/alex_the_brave
Reddit - https://old.reddit.com/r/brave_browser/comments/g79s8n/is_there_a_way_to_remove_the_tip_button/
Vimeo - https://vimeo.com/bravelaurenwagsLogged #12968 Unable to test Vimeo due to this issue |
Description
Make Brave Rewards extension non-persistent via manifest key
persistent: false
, allowing extension to be unloaded when not in active use (read when the popup UI is not shown). This should save one process and ~19MB of RAM.Steps to Reproduce
Actual result:
Expected result:
Brave Rewards process is not running, since the popup is closed.
Reproduces how often:
All the time.
Brave version (brave://version info)
Brave 0.69.132 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Linux
JavaScript V8 7.7.299.11
Flash (Disabled)
User Agent Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/77.0.3865.90 Safari/537.36
Command Line /usr/bin/brave-browser-stable --enable-dom-distiller --disable-domain-reliability --disable-chrome-google-url-tracking-client --no-pings --extension-content-verification=enforce_strict --extensions-install-verification=enforce --enable-features=NewExtensionUpdaterService,WebUIDarkMode,SimplifyHttpsIndicator --disable-features=AudioServiceOutOfProcess,AutofillServerCommunication,LookalikeUrlNavigationSuggestionsUI,UnifiedConsent --flag-switches-begin --flag-switches-end
Executable Path /opt/brave.com/brave/brave-browser
Profile Path /home/anton/.config/BraveSoftware/Brave-Browser/Default
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
The following bugs are related:
It's about what happens when the Rewards are entirely disabled
Different extension, the same challenges
The text was updated successfully, but these errors were encountered: