Closed Bug 581078 Opened 12 years ago Closed 12 years ago

Get rid of Utils.activeTab to avoid iterating over all Tabs

Categories

(Firefox Graveyard :: Panorama, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Mardak, Assigned: Mardak)

References

Details

Attachments

(1 file)

For now we can grab the BrowserTab using .tabcandyBrowserTab, but that should go away with bug 580937.
Attached patch v1Splinter Review
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #459481 - Flags: review?(ian)
http://hg.mozilla.org/users/edward.lee_engineering.uiuc.edu/tabcandy-central/rev/41ac0653ca60
Directly access gBrowser.selectedTab and compare other raw tabs against it until BrowserTab goes away. Get the reference to the BrowserTab instead of iterating over Tabs through .tabcandyBrowserTab.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Attachment #459481 - Flags: review?(ian) → feedback?(ian)
Comment on attachment 459481 [details] [diff] [review]
v1

Looks good to me
Attachment #459481 - Flags: feedback?(ian) → feedback+
Mass moving all Tab Candy bugs from Mozilla Labs to Firefox::Tab Candy.  Filter the bugmail spam with "tabcandymassmove".
Product: Mozilla Labs → Firefox
Target Milestone: -- → ---
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.