RunWatchdog should not fire when a debugger is present
Categories
(Core :: XPCOM, defect, P5)
Tracking
()
People
(Reporter: mayhemer, Unassigned)
References
(Blocks 1 open bug)
Details
![]() |
Reporter | |
Comment 2•9 years ago
|
||
![]() |
Reporter | |
Comment 4•9 years ago
|
||
Updated•7 years ago
|
Updated•3 years ago
|
Comment 5•3 years ago
|
||
So IIUC IsDebuggerPresent
is a WinAPI function we have no general equivalent for. It checks if there is an external debugger attached (probably Visual Studio?).
There is an easy workaround for this: just set the pref toolkit.asyncshutdown.crash_timeout
to something much higher than the default. Checking IsDebuggerPresent
would be easy to do but is it worth to have a special case here just for Windows? If someone is debugging shutdown I hope they know how it works in general...
Comment 6•3 years ago
|
||
I don't really see the harm in adding a special case for Windows to the code, it seems straightforward enough to add by setting the timeout to INT32_MAX
if ::IsDebuggerPresent()
, and like it'd be unlikely to cause problems. Might be worth asking if it would be handy for folks in #windev on matrix?
Comment 8•3 years ago
|
||
By now no response on #windev. If someone stumbles again over this, we are happy to look at a patch.
Description
•