The tabbed browser's removeTab method has an aDisableUndo argument but that only disables the tabbrowser's undo and not the session restore undo close tab. This has become significant because of bug 530735 which allows the session restore undo close tab to operate on the tab.
Created attachment 426231 [details] [diff] [review] Proposed patch The session store code already expects a UIEvent with a detail anyway.
Misak, I don't suppose that this is something that you can write a test for?
Comment on attachment 426231 [details] [diff] [review] Proposed patch I think i should add code to handle aDisableUndo in sessionrestore.
Attachment #426231 - Flags: review?(misak) → review+
It's already there in line 419 of nsSessionStore.js: if (!aEvent.detail) this.onTabClose(win, aEvent.originalTarget); Its purpose was originally to deal with switching tabs between browsers, where you also want to disable undo.
Ah, I'm starring to the code two days to find how it should work. Thanks for explanation :)
(In reply to comment #5) > Ah, I'm starring to the code two days to find how it should work. Thanks for > explanation :) No, starring is what you do to non-green Tinderboxes ;-) (Mind you, many developers seem to be staring at the non-green Tinderboxes, not that it helps!)
Whoops, forgot to update the bug when I pushed the patch a couple of weeks ago. Pushed changeset b4e566c1043b to comm-central.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.