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

50-80ms delay when closing a tab #7395

Closed
bradleyrichter opened this issue Feb 27, 2017 · 3 comments · Fixed by #9191
Closed

50-80ms delay when closing a tab #7395

bradleyrichter opened this issue Feb 27, 2017 · 3 comments · Fixed by #9191

Comments

@bradleyrichter
Copy link
Contributor

bradleyrichter commented Feb 27, 2017

Test plan

  1. Have a large number of bookmarks
  2. Open a few of those bookmarks
  3. Close each, one at a time. Should close immediately

for more info, see #9191 (comment)

Original issue description

Adding to the appearance of sluggishness is the delay seen when closing a tab.

Click on the close button.

Notice how long it takes before it closes after mouse-up event.

Brave: 0.13.5
rev: 7373a01
Muon: 2.56.7
libchromiumcontent: 56.0.2924.87
V8: 5.6.326.50
Node.js: 7.4.0
Update Channel: dev
os.platform: darwin
os.release: 16.3.0
os.arch: x64

@bradleyrichter bradleyrichter added this to the 1.0.0 milestone Feb 27, 2017
@netinhoteixeira
Copy link

I have the same issue.

@netinhoteixeira
Copy link

Brave 0.15.314
rev 75ffa36
Muon 3.0.202
libchromiumcontent 58.0.3029.110
V8 5.8.283.38
Node.js 7.9.0
Update Channel dev
os.platform darwin
os.release 16.6.0
os.arch x64

@luixxiul luixxiul added the perf label Jun 2, 2017
@bradleyrichter
Copy link
Contributor Author

@ayumi is finishing up a fix on this. Will land very soon. See #9191.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.