Closed Bug 338040 Opened 15 years ago Closed 14 years ago

Get rid of the tryToClose pseudo-API

Categories

(Toolkit :: Startup and Profile System, defect)

defect
Not set
minor

Tracking

()

RESOLVED WONTFIX

People

(Reporter: zeniko, Unassigned)

References

Details

Attachments

(1 file)

AFAICS the tryToClose methods are only needed at shutdown. But for this case Toolkit introduced the quit-application-requested global notification. Is there any reason for not getting rid of tryToClose and only rely on the notification?
Since this touches several source modules, who else should review this patch?
Attachment #223363 - Flags: first-review?(mconnor)
BTW: This patch should prevent SessionStore from losing window data if an extension calls nsIAppStartup::Quit without calling canQuitApplication.
Attachment #223363 - Flags: first-review?(mconnor)
Depends on: 386002
(In reply to comment #2)
> BTW: This patch should prevent SessionStore from losing window data if an
> extension calls nsIAppStartup::Quit without calling canQuitApplication.

Looks like this is what happened to the "Restart Firefox extension" (https://addons.mozilla.org/en-US/firefox/addon/1249). I've told the author about it.
With the removal of nsCloseAllWindows.js, the relevant code has taken a different path. -> WONTFIX
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → WONTFIX
Component: XRE Startup → Startup and Profile System
QA Contact: xre.startup → startup
You need to log in before you can comment on or make changes to this bug.