[windows] Include the return value of CoInitializeEx in crashes

WorkerThreadCOMDelegate fails to inititialize the thread as a COM
single-threaded apartment from time to time. Many of these are due to
OOM or ATOM exhaustion, and are handled by Chrome's
`OnResourceExhausted` handler, but some are not. Include the return
value from CoInitializeEx in a crash key in this case so that we can try
to gain a bit more understanding of what is actually happening here.

Bug: 40074523
Change-Id: Ie9b305ed7227cd9c54f76d4e79f9f38ca4a88dc9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5970334
Auto-Submit: Greg Thompson <grt@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1374581}


CrOS-Libchrome-Original-Commit: 26198b17db908c6b201889e5b5cfd5537edbd575

Author: GregTho

base/task/thread_pool/pooled_single_thread_task_runner_manager.cc

@@ -41,6 +41,7 @@
 #if BUILDFLAG(IS_WIN)
 #include <windows.h>
 
+#include "base/debug/crash_logging.h"
 #include "base/win/scoped_com_initializer.h"
 #endif  // BUILDFLAG(IS_WIN)
 
@@ -323,8 +324,20 @@ class WorkerThreadCOMDelegate : public WorkerThreadDelegate {
 
     scoped_com_initializer_ = std::make_unique<win::ScopedCOMInitializer>();
 
-    // CHECK to make sure this COM thread is initialized correctly in an STA.
-    CHECK(scoped_com_initializer_->Succeeded());
+    // Make sure this COM thread is initialized correctly in an STA. The thread
+    // would be in the default MTA state upon failure, which would mean any
+    // other MTA thread could service calls invoked by COM on objects living in
+    // this apartment.
+    if (!scoped_com_initializer_->Succeeded()) {
+      // Collect the reason when CoInitializeEx fails. Classic OOM (or ATOM
+      // exhaustion) should lead to process death in ScopedCOMInitializer, but
+      // other failures will leak out. Collect the failure codes in an effort to
+      // understand whether or not these failures are actionable; see
+      // https://crbug.com/40074523.
+      SCOPED_CRASH_KEY_NUMBER("WorkerThreadCOMDelegate", "hr",
+                              scoped_com_initializer_->hr());
+      NOTREACHED();
+    }
   }
 
   RegisteredTaskSource GetWork(WorkerThread* worker) override {