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

Clearing history should clear url suggestions #10962

Closed
bbondy opened this issue Sep 15, 2017 · 4 comments
Closed

Clearing history should clear url suggestions #10962

bbondy opened this issue Sep 15, 2017 · 4 comments
Assignees

Comments

@bbondy
Copy link
Member

bbondy commented Sep 15, 2017

@NejcZdovc mind taking this? Please make sure it works across the branches. It might already work for sites splitting branch for example but need it fixed on 0.19.x too.

  • Did you search for similar issues before submitting this one?
    yes

  • Describe the issue you encountered:
    Clearing history in about:preferences (on demand or on shutdown) keeps url suggestions

  • Platform (Win7, 8, 10? macOS? Linux distro?):

  • Brave Version (revision SHA):
    0.18.29

  • Steps to reproduce:

    1. Make a backup of your profile
    2. Go to about:preferences
    3. Clear browsing history
    4. Restart the app
    5. Type a URL that's in your history into the URL bar, it should not show up as a suggestion.
  • Actual result:
    about:history doesn't have the entry, but the URL bar does.

  • Expected result:
    Suggestions should no longer show.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    yes

  • Is this an issue in the currently released version?
    yes

  • Can this issue be consistently reproduced?
    yes

@NejcZdovc
Copy link
Contributor

@bbondy only in 0.21.x we added remove functionality for bloodhound. Should we trigger complete re-initialization of bloodhound when clear is triggered manually?

On shutdown there shouldn't be any problems

@bbondy
Copy link
Member Author

bbondy commented Sep 18, 2017

This is even if you close the browser, so something else seems to be going on. I'll take a look since you're on ledger work.

@bbondy bbondy assigned bbondy and unassigned NejcZdovc Sep 18, 2017
@bbondy
Copy link
Member Author

bbondy commented Sep 18, 2017

Actually this is invalid / a dupe of something already fixed.
If you close the browser it doesn't show up, it's just the in memory cache while it is open.

@bbondy bbondy added the invalid label Sep 18, 2017
@bbondy
Copy link
Member Author

bbondy commented Sep 18, 2017

Dupe of #10723

@bbondy bbondy closed this as completed Sep 18, 2017
@bbondy bbondy removed this from the 0.19.x (Beta Channel) milestone Sep 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants