win: Fix incorrect thread suspend count due to ScopedProcessSuspend

util/win/exception_handler_server.cc

Index: util/win/exception_handler_server.cc
diff --git a/util/win/exception_handler_server.cc b/util/win/exception_handler_server.cc
index 547087e04be714bcf41feda9c791c1bb489dbdfe..c736b016f30a65f601b1d601cf4266ab96a060ac 100644
--- a/util/win/exception_handler_server.cc
+++ b/util/win/exception_handler_server.cc
@@ -27,6 +27,7 @@
 #include "util/misc/tri_state.h"
 #include "util/misc/uuid.h"
 #include "util/win/registration_protocol_win.h"
+#include "util/win/scoped_process_suspend.h"
 
 namespace crashpad {
 
@@ -408,6 +409,8 @@ void __stdcall ExceptionHandlerServer::OnDumpEvent(void* ctx, BOOLEAN) {
   internal::ClientData* client = reinterpret_cast<internal::ClientData*>(ctx);
   base::AutoLock lock(*client->lock());
 
+  ScopedProcessSuspend suspend(client->process());

[Mark Mentovai, 2015/09/09 18:57:27]

I think this worked better in CrashReportExceptionHandler. I don’t think it
should be up to the base to decide whether suspension is appropriate or not.
That’s a decision for the delegate.

[scottmg, 2015/09/09 19:13:25]

OK, done.

+
   // Capture the exception.
   unsigned int exit_code = client->delegate()->ExceptionHandlerServerException(
       client->process(), client->exception_information_address());