Closed Bug 822914 Opened 9 years ago Closed 9 years ago
Browser .selected Tab getter efficiency
And replace (aTab == this.selectedTab) with aTab.selected. .selectedTab getter seems more expensive than .mCurrentTab.
Comment on attachment 693717 [details] [diff] [review] patch The tab.selected changes are fine, but we should make selectedTab fast rather than using mCurrentTab.
So maybe selectedTab getter could return mCurrentTab directly, and use tabContainer.selectedItem instead when necessary?
This sounds reasonable.
I'm not sure whether there is some risk. I'm testing it in my profile.
Attachment #693717 - Attachment is obsolete: true
It just works.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
You need to log in before you can comment on or make changes to this bug.