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

Follow up to #7240, Bookmark import reduces performance when trying to open bookmarks from bookmark toolbar #9651

Closed
srirambv opened this issue Jun 21, 2017 · 5 comments

Comments

@srirambv
Copy link
Collaborator

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

  • Describe the issue you encountered:
    Browser performance reduces when large amount of bookmarks are imported

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

  • Brave Version (revision SHA):
    Brave 0.17.4
    rev 39c7a53
    Muon 4.1.2

  • Steps to reproduce:

    1. Import a large bookmark HTML file
    2. Open a new tab and visit any site/the one that is bookmarked
    3. Bookmark toolbar -> Imported folder -> Open bookmark folder
  • Actual result:
    Bookmark import reduces performance when trying to open bookmarks from bookmark toolbar

  • Expected result:
    Imported bookmarks should not impact browser performance

  • 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

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:

  • Any related issues:
    Browser performance reduces when large amount of bookmarks are imported #7240
    Please change milestone if required

@srirambv srirambv added this to the 0.19.x (Nightly Channel) milestone Jun 21, 2017
@LaurenWags
Copy link
Member

+1 from Community, user has approx 1000 bookmarks. https://community.brave.com/t/rearranging-bookmarks/4666

@alexwykoff alexwykoff added the needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. label Jul 18, 2017
@alexwykoff
Copy link
Contributor

@srirambv can you confirm this is fixed in 0.19.x?

@darkdh
Copy link
Member

darkdh commented Jul 27, 2017

#10054 has specified the cause for this problem

@srirambv
Copy link
Collaborator Author

@alexwykoff Performance is much much better than before. Its not completely smooth usage. I still see some lag in scrolling through bookmarks in nested folders and see quite a high CPU usage

@alexwykoff
Copy link
Contributor

this is covered by #10054

@luixxiul luixxiul removed this from the 0.19.x (Beta Channel) milestone Aug 9, 2017
@luixxiul luixxiul removed the needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. label Aug 9, 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

5 participants