Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

busyP errors when toggling payments off/on and leaving off #11684

Closed
LaurenWags opened this issue Oct 26, 2017 · 1 comment
Closed

busyP errors when toggling payments off/on and leaving off #11684

LaurenWags opened this issue Oct 26, 2017 · 1 comment

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Oct 26, 2017

Description

Steps to Reproduce

  1. launch 0.18.36 and enable payments via about:preferences#payments
  2. visit several websites and ensure they're appearing under the ledger table
  3. disable payments via the "On/Off" switch under about:preferences#payments
  4. switch to 0.19.67 while using ledger_verbose=true
  5. go to preferences and try to toggle payments on/off multiple times
  6. leave toggle set to off for a minute or two

Actual result:
After leaving toggle off for a minute or two, 'busyP' errors will display, payments toggle will be stuck in off position. Additionally, @srirambv reported his entire browser was frozen after getting those errors.

Expected result:
No errors, toggle/browser should not freeze.

Reproduces how often: [What percentage of the time does it reproduce?] 100%

Brave Version

about:brave info:
Brave | 0.19.67
rev | b3a897e
Muon | 4.5.9

Reproducible on current live release:
no

Additional Information

https://gist.github.com/LaurenWags/effdd07aa2d5eaf95e96064915111e86#file-busyp-txt-L1289

Found on MacOS, reproduced by @srirambv on Windows

@LaurenWags
Copy link
Member Author

Found while testing #11611 and #11626

NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Nov 6, 2017
Resolves brave#11703
Resolves brave#11684

Auditors:

Test Plan:
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Nov 6, 2017
Resolves brave#11703
Resolves brave#11684

Auditors:

Test Plan:
syuan100 pushed a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
Resolves brave#11703
Resolves brave#11684

Auditors:

Test Plan:
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.