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

Tracking: Critical performance tracking issue #9311

Closed
24 of 35 tasks
bsclifton opened this issue Jun 7, 2017 · 9 comments
Closed
24 of 35 tasks

Tracking: Critical performance tracking issue #9311

bsclifton opened this issue Jun 7, 2017 · 9 comments
Assignees
Labels
addressed-with-brave-core Needs confirmation, but this issue may be resolved with Brave Core. perf priority/P2 Crashes. Loss of data. Severe memory leak. project-tracking

Comments

@bsclifton
Copy link
Member

bsclifton commented Jun 7, 2017

The goal of this issue is to track our most painful performance impacting issues

General slowness

Tab switching slowness

URL bar

Tab close speed

Page navigation

About pages

  • Use bloodhound for about:bookmarks
  • Query bloodhound for about:history

Worker threads

Auditing / instrumentation

Bookmark / sites related

Reflows

Slow startup time

Battery life

@bsclifton bsclifton added this to the 0.17.x (Frozen, only critical adds from here) milestone Jun 7, 2017
@bsclifton bsclifton self-assigned this Jun 7, 2017
@bsclifton bsclifton modified the milestones: 0.16.x (Frozen, only critical adds from here), 0.17.x (Frozen, only critical adds from here) Jun 7, 2017
@ayumi
Copy link
Contributor

ayumi commented Jun 7, 2017

Tab switching slowness should be improved by brave/muon#208 (browser-laptop #9313).

@bsclifton
Copy link
Member Author

@ayumi awesome! We're currently on 4.0.0 which doesn't have the fix... so we'll need a new build. The work you did with #9279 helps a great deal if the tab isn't loaded yet (ex: after you open browser OR if tab is unloaded due to tab discarding)

@luixxiul luixxiul added the perf label Jun 8, 2017
@alexwykoff alexwykoff modified the milestones: 0.17.x (Frozen, only critical adds from here), 0.16.x (Frozen, only critical adds from here) Jun 9, 2017
@bbondy bbondy modified the milestones: 0.18.x (Frozen, only critical adds from here), 0.17.x (Frozen, only critical adds from here) Jun 11, 2017
@bbondy bbondy changed the title Critical performance tracking issue Tracking: Critical performance tracking issue Jun 11, 2017
@alexwykoff alexwykoff modified the milestones: 0.19.x (Nightly Channel), 0.18.x (Developer Channel) Jun 23, 2017
@ayumi
Copy link
Contributor

ayumi commented Jul 5, 2017

I've been experiencing an intense browser UI / chrome slowness (browser with 3K bookmarks):

I have a bookmarks toolbar with stuff in it, and when I hover over it takes 2000 ms before buttons get the hover effect (highlighting to suggest clickability). Tabs lag similarly.
This slowness doesn't appear to be in the reducers.

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Jul 5, 2017

@ayumi on the master or 0.17? for master (0.19) this PR should fix perf problems #9713

@ayumi
Copy link
Contributor

ayumi commented Jul 5, 2017

@NejcZdovc On both master (0562bf4) and the current preview release.

Happening just on master (0562bf4).

@NejcZdovc
Copy link
Contributor

for master I have a PR up, but for the current release it's not relevant, because we merged redux for bookmarks in 0.19

@ayumi
Copy link
Contributor

ayumi commented Jul 5, 2017

@NejcZdovc Sorry I checked again and the latest preview is okay ( 0.17.13 rev f7242c7) – the UI slowness was just in master.

@NejcZdovc
Copy link
Contributor

Make sense, so that PR should fix this slowness. It's already done, but we want to improve one more thing

@alexwykoff alexwykoff modified the milestones: 0.20.x (Developer Channel), 0.19.x (Beta Channel) Jul 18, 2017
@bsclifton bsclifton added the hackathon Legacy label for a hackaton. label Jul 26, 2017
@alexwykoff alexwykoff removed this from the 0.20.x (Developer Channel) milestone Aug 8, 2017
@ghost ghost added the Epic label Aug 21, 2017
@NejcZdovc NejcZdovc removed the Epic label Nov 7, 2017
@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@bsclifton bsclifton added the priority/P2 Crashes. Loss of data. Severe memory leak. label Nov 27, 2017
@bsclifton bsclifton modified the milestones: Triage Backlog, Backlog (Prioritized) Nov 27, 2017
@bsclifton bsclifton removed the hackathon Legacy label for a hackaton. label Nov 27, 2017
@bsclifton bsclifton removed this from the Backlog (Prioritized) milestone Aug 23, 2018
@bsclifton bsclifton added the addressed-with-brave-core Needs confirmation, but this issue may be resolved with Brave Core. label Aug 23, 2018
@bsclifton
Copy link
Member Author

Most (if not all) of these issues should be addressed with brave-core

For folks that want to try specific issues with an experimental release of our new product, you can download a recent release and try it out:
https://github.com/brave/brave-browser/releases

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
addressed-with-brave-core Needs confirmation, but this issue may be resolved with Brave Core. perf priority/P2 Crashes. Loss of data. Severe memory leak. project-tracking
Projects
None yet
Development

No branches or pull requests

6 participants