Closed
Bug 596806
Opened 14 years ago
Closed 14 years ago
Pass tab instead of browser to onTabAdd, onTabRemove
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
FIXED
Firefox 4.0b7
People
(Reporter: zpao, Assigned: zpao)
References
Details
Attachments
(1 file)
5.38 KB,
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
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)
Assignee | ||
Comment 1•14 years ago
|
||
Easy, and test-free (just an internal API change). I ran the sessionstore tests to make sure nothing broke, so we should be good.
Comment 2•14 years ago
|
||
Comment on attachment 475708 [details] [diff] [review] Patch v0.1 r=me, yay tabs!
Attachment #475708 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 3•14 years ago
|
||
Pushed http://hg.mozilla.org/mozilla-central/rev/957c89c4eac3
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b7
You need to log in
before you can comment on or make changes to this bug.
Description
•