Breakpad for windows - disable DumpProcessWithoutCrash to see if UMA

components/breakpad/app/breakpad_win.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/breakpad/app/breakpad_win.h"
 
 #include <windows.h>
 #include <shellapi.h>
 #include <tchar.h>
 #include <userenv.h>
@@ skipping 555 matching lines @@
   g_dumphandler_no_crash = new google_breakpad::ExceptionHandler(temp_dir,
       &FilterCallbackWhenNoCrash,
       &DumpDoneCallbackWhenNoCrash,
       NULL,
       // Set the handler to none so this handler would not be added to
       // |handler_stack_| in |ExceptionHandler| which is a list of exception
       // handlers.
       google_breakpad::ExceptionHandler::HANDLER_NONE,
       dump_type, pipe_name.c_str(), custom_info);
 
-  base::debug::SetDumpWithoutCrashingFunction(&DumpProcessWithoutCrash);
+  // TODO(rtenneti): http://crbug.com/391165. Undo this change after getting
+  // data. Disabled DumpProcessWithoutCrash to see if UMA crash rate decreases.
+  // base::debug::SetDumpWithoutCrashingFunction(&DumpProcessWithoutCrash);
 
   if (g_breakpad->IsOutOfProcess()) {
     // Tells breakpad to handle breakpoint and single step exceptions.
     // This might break JIT debuggers, but at least it will always
     // generate a crashdump for these exceptions.
     g_breakpad->set_handle_debug_exceptions(true);
 
 #ifndef _WIN64
     if (process_type != L"browser" &&
         !GetBreakpadClient()->IsRunningUnattended()) {
       // Initialize the hook TerminateProcess to catch unexpected exits.
       InitTerminateProcessHooks();
     }
 #endif
   }
 }
 
 }  // namespace breakpad