Bug 1696620 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

The WER exception module is only currently writing out a minimum set of crash annotations, these should be augmented with the following which it should be possible to extract from without Gecko:
* CrashTime: this is just a call to `time()`.
* Memory measurements: these include AvailablePageFile, AvailablePhysicalMemory, TotalPageFile, TotalPhysicalMemory and SystemMemoryUsePercentage. See the code in [AnnotateMemoryStatus()](https://searchfox.org/mozilla-central/rev/bff70a63cff89284426d82ded6469ed708ef68b7/toolkit/crashreporter/nsExceptionHandler.cpp#778-796). There is no point in recording AvailableVirtualMemory and TotalVirtualMemory because we're taking the measurements outside of the crashed process.
* Info about the graphics driver: AdapterDeviceID, AdapterDriverVersion, AdapterSubsysID and AdapterVendorID. These might be harder to extract (I need to study the widget code) but would be very nice to add given that we expect crashes to be coming from graphics drivers too.
* CPUMicrocodeVersion and MemoryErrorCorrection: nice to have but not necessary
The WER exception module is only currently writing out a minimum set of crash annotations, these should be augmented with the following which it should be possible to extract from without Gecko:
* ~~CrashTime: this is just a call to `time()`.~~
* StartupTime: this needs to be computed from `GetProcessTimes()`
* Memory measurements: these include AvailablePageFile, AvailablePhysicalMemory, TotalPageFile, TotalPhysicalMemory and SystemMemoryUsePercentage. See the code in [AnnotateMemoryStatus()](https://searchfox.org/mozilla-central/rev/bff70a63cff89284426d82ded6469ed708ef68b7/toolkit/crashreporter/nsExceptionHandler.cpp#778-796). There is no point in recording AvailableVirtualMemory and TotalVirtualMemory because we're taking the measurements outside of the crashed process.
* Info about the graphics driver: AdapterDeviceID, AdapterDriverVersion, AdapterSubsysID and AdapterVendorID. These might be harder to extract (I need to study the widget code) but would be very nice to add given that we expect crashes to be coming from graphics drivers too.
* CPUMicrocodeVersion and MemoryErrorCorrection: nice to have but not necessary

Back to Bug 1696620 Comment 0