Instead of Tabbar.showOnlyTheseTabs, it can be part of tabbrowser.
Assuming bug 580847 is fixed to move the tabs to the right places, there shouldn't be a need for the current functionality of reordering tabs when showing only those tabs.
Created attachment 459970 [details] [diff] [review]
Created attachment 459972 [details] [diff] [review]
Seems like Raymond rolled the show functionality into UIManager:
(In reply to comment #3)
> Seems like Raymond rolled the show functionality into UIManager:
Oh, sorry. I saw the comment 2 in bug 581267 and miss this one.
Move and simplify showOnlyTheseTabs functionality into tabbrowser now that there's no reordering happening on show.
Call updateTabBarForActiveGroup so that Group is correctly set for this.
Comment on attachment 459972 [details] [diff] [review]
Looks good except for this line:
"this" won't be set up properly.
See comment 6.
Mass moving all Tab Candy bugs from Mozilla Labs to Firefox::Tab Candy. Filter the bugmail spam with "tabcandymassmove".
Added to the list of methods on tabbrowser here:
Linked from Firefox 4 for developers.
sheppy, sorry for the confusion, but this bug originally landed as part of the tabcandy-central branch, and the code that landed in m-c is in bug 582116 where there ended up not being a showAllTabs.
Removed showAllTabs from the docs; thanks!