649908 - showOnlyTheseTabs() is being called unnecessarily often

Status: RESOLVED WONTFIX

(Firefox Graveyard :: Panorama, defect)

Description

[Frank Yan (:fryn)]

One of many ways to reproduce this:
1. Go to panorama view.
2. Drag to create a new group.
3. Click on the new tab icon in the new group, which zooms into that tab. This calls showOnlyTheseTabs(), which is sensible.
4. Ctrl+t or Cmd+t to open another tab in the new group. Why does this need to call showOnlyTheseTabs() again?
5. Repeat step 4 as many times as you'd like.

This should be optimized, IMHO.

Comment 1

[Frank Yan (:fryn)]

How I discovered this:

I thought that showOnlyTheseTabs() only was called when switching between groups, so I put |this.tabContainer._unlockTabSizing();| at the beginning of that function, and it resulted in unlocking way too often.