Closed Bug 596806 Opened 10 years ago Closed 10 years ago

Pass tab instead of browser to onTabAdd, onTabRemove

Categories

(Firefox :: Session Restore, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 4.0b7

People

(Reporter: zpao, Assigned: zpao)

References

Details

Attachments

(1 file)

It's going to make work I'm doing in bug 586068 easier and more efficient (instead of getting a browser and having to find a tab from it, I'll just have the tab right away)
Attached patch Patch v0.1Splinter Review
Easy, and test-free (just an internal API change). I ran the sessionstore tests to make sure nothing broke, so we should be good.
Assignee: nobody → paul
Status: NEW → ASSIGNED
Attachment #475708 - Flags: review?(dietrich)
Comment on attachment 475708 [details] [diff] [review]
Patch v0.1

r=me, yay tabs!
Attachment #475708 - Flags: review?(dietrich) → review+
Pushed http://hg.mozilla.org/mozilla-central/rev/957c89c4eac3
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b7
Blocks: 597584
Blocks: 617987
You need to log in before you can comment on or make changes to this bug.