Closed
Bug 1051585
Opened 10 years ago
Closed 5 years ago
After rebooting windows with "firefox.exe -silent", firefox thought it crashed and show a session restore tab.
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
INCOMPLETE
People
(Reporter: richard.berger, Unassigned)
Details
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:31.0) Gecko/20100101 Firefox/31.0 (Beta/Release) Build ID: 20140716183446 Steps to reproduce: Restarting windows while firefox is open. Actual results: Windows rebooted. Firefox complains about an crash and shows session restore tab. Expected results: Windows rebooted. Firefox should open normal with the last tabs open. That worked just fine in Version 28. After upgrading to version 31, this bad behavior was introduced.
Comment 1•10 years ago
|
||
I don't think there's much we can do about that. When you shut down Windows, all open programs get a small amount of time to quit themselves. If they don't manage to do that, Windows just kills the process, leading to a dirty state of Firefox.
Reporter | ||
Comment 2•10 years ago
|
||
I am aware of that. But that worked for years flawless, until I upgraded to Firefox >=29. Thats the reason why I am reporting this. Something has changed to the bad in that versions.
Is there a way to simulate (script?) a Windows shutdown to firefox.exe? It would help to test without restarting Windows a ton of times.
Comment 4•10 years ago
|
||
Steven, any ideas of what's going on here and/or what might have changed after 28?
Component: Untriaged → Session Restore
Flags: needinfo?(smacleod)
Reporter | ||
Comment 5•10 years ago
|
||
fyi: I didn't test with version 29. I upgraded directly from version 28 to 31.
Comment 6•10 years ago
|
||
Off the top of my head I don't know of a particular change that would be causing this to fail now. I wonder if the SessionCheckpoints.json write isn't finishing in this case, even though session.js is, so we think we crashed? So maybe the CrashMonitor is involved? Tim / David, any thoughts on what could have changed?
Flags: needinfo?(ttaubert)
Flags: needinfo?(smacleod)
Flags: needinfo?(dteller)
Comment 7•10 years ago
|
||
We introduced the crash monitor with bug 888373 in Firefox 29 iirc. I have no idea about the Windows shutdown process but as you said, maybe only writing sessionCheckpoints.json fails or doesn't complete fast enough. If Windows just kills processes when shutting down then that's a problem and we couldn't do much about it. If it however sends a signal we might try hooking onto that and see what we can do here. It would help to reproduce and get a snapshot of what we managed to write to disk before shutdown.
Flags: needinfo?(ttaubert)
Comment 8•10 years ago
|
||
Richard, next time you need to reboot, can you check whether you have a file called sessionCheckpoints.json in your profile directory and, if so, attach it to this bug? It doesn't contain any private information.
Flags: needinfo?(dteller) → needinfo?(richard.berger)
Reporter | ||
Comment 9•10 years ago
|
||
Hello, I think I could identify the problem I had. To speed up firefox first start after a reboot, I had a shortcut to "firefox.exe -silent" in my autorun. I read about it here: https://support.mozilla.org/de/questions/762446 After disabling the shortcut, the problem was gone. To cross check I activated it again and nothing happens for a few boots. Than the problem returned on every boot. For now I disabled the shortcut. Unfortunately, firefox now needs nearly 30 seconds for its first start after a reboot.
Flags: needinfo?(richard.berger)
Updated•10 years ago
|
Summary: After rebooting windows, firefox tought it crashed and show a session restore tab. → After rebooting windows with "firefox.exe -silent", firefox thought it crashed and show a session restore tab.
Good catch. Cc-ing Robert, who wrote the code for "-silent".
Component: Session Restore → General
Flags: needinfo?(robert.strong.bugs)
Comment 11•10 years ago
|
||
I didn't implement silent... all I added was the handler in nsBrowserContentHandler.js back in 2006. As bug 324023 comment #3 states, back then xpfe still handled -silent but toolkit apps didn't even though we still set the reg values on win32 to use -silent for the same purpose.
Flags: needinfo?(robert.strong.bugs)
Comment 12•8 years ago
|
||
Richard, do you still see this problem?
Severity: normal → minor
Flags: needinfo?(richard.berger)
Comment 13•5 years ago
|
||
Unclear if this is still an issue, and there's been lots of code changes since this was reported. No recent activity, so I'll close as incomplete. Please re-open if this is still an issue and provide what details you can.
Status: UNCONFIRMED → RESOLVED
Closed: 5 years ago
Flags: needinfo?(richard.berger)
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•