1793904 - Reduce log verbosity of content process shutdown

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, task)

Description

[Jens Stutte [:jstutte]]

MaybeLogBlockShutdownDiagnostics currently prints out a NS_DEBUG_WARNING if MOZ_DIAGNOSTIC_ASSERT_ENABLED. This leads to quite verbose log files in treeherder during shutdown. We have now better support for crashstats through CrashReporter::AppendToCrashReportAnnotation(CrashReporter::Annotation::IPCShutdownState...) (see bug 1777198) so while there is value in having this information in treeherder logs we might want to limit it to DEBUG builds only or make it a normal log INFO message to make it at least less scary.

Comment 1

[Jens Stutte [:jstutte]]

Attachment: Bug 1793904 - Downgrade content process shutdown warnings to info. r?smaug

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dd8168ec6c99
Downgrade content process shutdown warnings to info. r=smaug

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/dd8168ec6c99