478750 - closing last tab using _endRemoveTab in tabbrowser.xml is synchronous and performance intensive

Status: RESOLVED WONTFIX

(Firefox :: Tabbed Browser, defect)

Description

[Mike Beltzner [:beltzner, not reading bugmail]]

Spun off from bug 476928 which itself was spawned from bug bug 473843 comment 10 and bug 473843 comment 11

Quoting from bug 473843:

> [it] seems to be that _endRemoveTab in tabbrowser.xml does some
> performance intensive scrolling (sync'd) when the last tab is removed

In bug 476928 Henrik suggests we "fix this in tabbbrowser" instead of using a suggested wallpaper fix (don't remove tabs starting from the end). I'm spinning this bug out because I'm

 - not sure we can fix this (as the scrolling behaviour is desired, I think)
 - entirely sure we don't want to block up fixing bug 476928

The purpose of this bug is to look at _endRemoveTab and see if there are ways to fix the synchronous performance issue that it contains.

Comment 1

[Dão Gottwald [:dao]]

(In reply to comment #0)
>  - not sure we can fix this (as the scrolling behaviour is desired, I think)

So do I. Making it asynchronous doesn't sound like it would make things snappier under normal circumstances (i.e. user closes a tab). And in not-so-normal circumstances, this can be optimized on the call side, as bug 476928 shows.

Unless there are better ideas how performance could be improved, I think this is wontfix.

Comment 2

[Wayne Mery (:wsmwk)]

(In reply to Dão Gottwald [:dao] from comment #1)
> 
> Unless there are better ideas how performance could be improved, I think
> this is wontfix.

time to close?

Comment 3

[Asa Dotzler [:asa]]

I think this is a wontfix.