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.

Sign up for GitHub

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

Jump to bottom

Fixed memory leak with rewards service URL fetchers #2213

Closed
bridiver opened this issue Nov 22, 2018 · 0 comments · Fixed by brave/brave-core#947
Closed

Fixed memory leak with rewards service URL fetchers #2213

bridiver opened this issue Nov 22, 2018 · 0 comments · Fixed by brave/brave-core#947
Assignees
@bridiver
Labels
bug feature/rewards QA/No release-notes/include
Milestone
0.57.x - Release ...

Comments

@bridiver
Copy link
Contributor

URLFetchers are not correctly deleted when they are complete
@bridiver bridiver self-assigned this Nov 22, 2018
@bridiver bridiver added the bug label Nov 22, 2018
bridiver added a commit to brave/brave-core that referenced this issue Nov 22, 2018
@bridiver
correctly delete fetchers
6aecd22 
fix brave/brave-browser#2213
@bridiver bridiver mentioned this issue Nov 22, 2018
Merged
18 tasks
@bbondy bbondy added this to the 0.58.x - Dev milestone Nov 22, 2018
@bbondy bbondy added the QA/No label Nov 22, 2018
@NejcZdovc NejcZdovc modified the milestones: 0.58.x - Dev, 0.57.x - Beta Nov 27, 2018
@bbondy bbondy changed the title rewards service url fetchers leak Fixed memory leak with rewards service url fetchers leak Dec 4, 2018
@bbondy bbondy changed the title Fixed memory leak with rewards service url fetchers leak Fixed memory leak with rewards service URL fetchers Dec 4, 2018
@kjozwiak kjozwiak mentioned this issue Dec 7, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bridiver bridiver

Labels
bug feature/rewards QA/No release-notes/include
Projects
None yet
Milestone
0.57.x - Release (Release Channel)
Development

Successfully merging a pull request may close this issue.

correctly delete fetchers brave/brave-core
4 participants
@bridiver @bbondy @rebron @NejcZdovc