Closed
Bug 334839
Opened 19 years ago
Closed 18 years ago
Update Session-Restore to use new TabOpen notification
Categories
(Firefox :: Tabbed Browser, enhancement)
Tracking
()
RESOLVED
FIXED
Firefox 2 beta1
People
(Reporter: dietrich, Assigned: dietrich)
References
Details
(Keywords: fixed1.8.1, Whiteboard: [swag: .2d])
Attachments
(1 file)
|
3.40 KB,
patch
|
mconnor
:
review+
mconnor
:
approval1.8.1+
|
Details | Diff | Splinter Review |
Once the tab changes land, update session-restore to use the new notifications.
|
Assignee | |
Updated•19 years ago
|
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•19 years ago
|
Whiteboard: [swag: 1d]
|
|
Assignee | |
Comment 1•18 years ago
|
||
I tested Seth's patch and it seems to work fine. Once it lands, it's trivial to make this change.
Whiteboard: [swag: 1d] → [swag: .2d]
|
|
Assignee | |
Updated•18 years ago
|
Target Milestone: Firefox 2 alpha2 → Firefox 2 beta1
|
|
Assignee | |
Comment 2•18 years ago
|
||
I left the onTabClose and onTabRemove methods separate as onTabRemove is validly used on it's own.
Attachment #227189 -
Flags: review?(mconnor)
Attachment #227189 -
Flags: approval1.8.1?
|
||
Comment 3•18 years ago
|
||
Comment on attachment 227189 [details] [diff] [review]
uses new tab events
r+a=me
Attachment #227189 -
Flags: review?(mconnor)
Attachment #227189 -
Flags: review+
Attachment #227189 -
Flags: approval1.8.1?
Attachment #227189 -
Flags: approval1.8.1+
|
|
Assignee | |
Updated•18 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•