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

With many tabs, startup is very slow #11424

Closed
ayumi opened this issue Oct 10, 2017 · 3 comments
Closed

With many tabs, startup is very slow #11424

ayumi opened this issue Oct 10, 2017 · 3 comments
Assignees
@petemill
Labels
perf priority/P3 Major loss of function. QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/exclude
Milestone
0.22.x Release 3 ...

Comments

@ayumi
Copy link
Contributor

ayumi commented Oct 10, 2017 •
edited
Loading

Description

On a user profile which has 100 unique tabs, app startup time is significantly slower vs startup with a fresh profile.
Total time to start, type a URL in the nav bar and navigate was ~7500 ms vs 4000 ms.

Steps to Reproduce

Automated test (feature/perf-tests branch): 648bafd#diff-e5c305031ff191e0f8d1fb021abeb435

  1. Start Brave
  2. Open 100 tabs each with unique URLs
  3. Close Brave
  4. Open Brave. It should remember the 100 tabs.
  5. Focus the URL bar, type a URL and hit enter to navigate.

Actual result:
Much slower with many tabs

Expected result:
Similar speed to a fresh profile

Reproduces how often:
Always in the perf tests

Brave Version

master 29bdbd5

Also repro'd on

Brave | 0.18.36
rev | 7ab85e9
Muon | 4.3.22
libchromiumcontent | 61.0.3163.79
V8 | 6.1.534.32
Node.js | 7.9.0
Update Channel | dev
OS Platform | macOS
OS Release | 15.6.0
OS Architecture | x64

Reproducible on current live release:

Additional Information

screen shot 2017-10-10 at 20 20 41
@ayumi ayumi added the perf label Oct 10, 2017
@ayumi ayumi added this to the 0.20.x (Developer Channel) milestone Oct 10, 2017
@ayumi ayumi self-assigned this Oct 18, 2017
@alexwykoff alexwykoff modified the milestones: 0.20.x (Beta Channel), 0.21.x (Developer Channel) Oct 24, 2017
@bbondy bbondy modified the milestones: 0.21.x (Developer Channel), 0.20.x (Beta Channel), Backlog Oct 25, 2017
@cezaraugusto cezaraugusto added priority/P4 Minor loss of function. Workaround usually present. priority/P3 Major loss of function. and removed priority/P4 Minor loss of function. Workaround usually present. labels Nov 21, 2017
@NejcZdovc NejcZdovc modified the milestones: Triage Backlog, Backlog (Prioritized) Nov 21, 2017
@petemill petemill assigned petemill and unassigned ayumi Feb 1, 2018
@petemill petemill modified the milestones: Backlog (Prioritized), 0.21.x (Beta Channel) Feb 1, 2018
@petemill
Copy link
Member

Attacking part (most?) of this with #13216

@petemill petemill mentioned this issue Feb 24, 2018
Closed
26 tasks
@petemill
Copy link
Member

Closing in favor of the more detailed #13216

@kjozwiak kjozwiak modified the milestones: 0.22.x (Beta Channel), 0.22.x Release 2 Mar 27, 2018
@petemill petemill modified the milestones: 0.22.x Release 2 (Beta Channel), 0.22.x Release 3 Apr 6, 2018
This was referenced May 3, 2018
Closed
Closed
@srirambv srirambv mentioned this issue May 3, 2018
Closed
@GeetaSarvadnya
Copy link
Collaborator

GeetaSarvadnya commented May 7, 2018 •
edited by btlechowski
Loading

Tested by loading win3-tabs105 in Roaming profile. 3 windows with 105 tabs in each window. There is no performance lag, click on each tab opens the website immediately. Click on new tab with new website is also opens website quickly.

Verified on Windows x64

  • 0.22.702 e4a853d
  • libchromiumcontent 66.0.3359.139
  • muon: 6.0.7

Verified with macOS 10.12.6 using

  • 0.22.703 903b8d0
  • muon 6.0.8
  • libchromiumcontent 66.0.3359.139

Verified on Ubuntu 17.10 x64

  • 0.22.703 903b8d0
  • libchromiumcontent 66.0.3359.139
  • muon: 6.0.8

This was referenced May 8, 2018
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@petemill petemill

Labels
perf priority/P3 Major loss of function. QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/exclude
Projects
None yet
Milestone
0.22.x Release 3 (Release Channel)
Development

No branches or pull requests