Always set bad_message_reason when logging bad message dumps

content/browser/bad_message.cc

Index: content/browser/bad_message.cc
diff --git a/content/browser/bad_message.cc b/content/browser/bad_message.cc
index 487742be1a893d8a209e251052e022ef92c4bcf9..ee8fba472bf1bbf39fb37334d0cdb4e00858a5be 100644
--- a/content/browser/bad_message.cc
+++ b/content/browser/bad_message.cc
@@ -33,8 +33,6 @@ void ReceivedBadMessageOnUIThread(int render_process_id,
   if (!host)
     return;
 
-  LogBadMessage(reason);
-
   // A dump has already been generated by the caller. Don't generate another.
   host->ShutdownForBadMessage(
       RenderProcessHost::CrashReportMode::NO_CRASH_DUMP);
@@ -49,8 +47,9 @@ void ReceivedBadMessage(RenderProcessHost* host, BadMessageReason reason) {
 }
 
 void ReceivedBadMessage(int render_process_id, BadMessageReason reason) {
-  // We generate a crash dump here since generating one after posting the UI
-  // thread is mostly useless.
+  // We generate a crash dump here since generating one after posting to the UI
+  // thread is less useful.
+  LogBadMessage(reason);
   base::debug::DumpWithoutCrashing();
 
   if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {