win: Move Crashpad all into chrome.exe

chrome/app/chrome_crash_reporter_client.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 "chrome/app/chrome_crash_reporter_client.h"
 
 #include "base/command_line.h"
 #include "base/environment.h"
 #include "base/files/file_path.h"
+#include "base/files/file_util.h"
 #include "base/logging.h"
 #include "base/path_service.h"
 #include "base/strings/string_split.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/threading/thread_restrictions.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_result_codes.h"
 #include "chrome/common/crash_keys.h"
 #include "chrome/common/env_vars.h"
+#include "chrome/installer/util/browser_distribution.h"
 #include "chrome/installer/util/google_update_settings.h"
 #include "content/public/common/content_switches.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 
 #include "base/file_version_info.h"
 #include "base/win/registry.h"
 #include "chrome/installer/util/google_chrome_sxs_distribution.h"
 #include "chrome/installer/util/install_util.h"
@@ skipping 242 matching lines @@
   *version = PRODUCT_VERSION;
 }
 
 base::FilePath ChromeCrashReporterClient::GetReporterLogFilename() {
   return base::FilePath(CrashUploadList::kReporterLogFilename);
 }
 #endif
 
 bool ChromeCrashReporterClient::GetCrashDumpLocation(
     base::FilePath* crash_dir) {
+#if defined(OS_WIN)
+  // In order to be able to start crash handling very early, we do not rely on
+  // the PathService being available on Windows. See http://crbug.com/564398.
+  if (GetAlternativeCrashDumpLocation(crash_dir))

[Mark Mentovai, 2015/12/02 22:47:12]

This slurps a path out of BREAKPAD_DUMP_LOCATION. What uses that? Is it
expecting the directory to have a Breakpad-type structure?

[scottmg, 2015/12/03 01:40:15]

The only place I could find on Windows is
https://code.google.com/p/chromium/codesearch#chromium/src/tools/telemetry/te...
but now that I look deeper, it's actually launching crash_service/Breakpad, so I
probably shouldn't honour this, and it'll need more porting anyway.

+    return true;
+  base::FilePath result;
+  if (!base::GetLocalAppData(&result))
+    return false;
+  BrowserDistribution* dist = BrowserDistribution::GetDistribution();
+  result = result.Append(dist->GetInstallSubDir());
+  result = result.Append(chrome::kUserDataDirname);

[Mark Mentovai, 2015/12/02 22:47:12]

Need to manually look for --user-data-dir too?

[scottmg, 2015/12/03 01:40:15]

This is copied from
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/common/chro...
which didn't respect that.

+  result = result.Append(FILE_PATH_LITERAL("Crashpad"));
+
+  // TODO(bauerb): http://crbug.com/259796
+  base::ThreadRestrictions::ScopedAllowIO allow_io;
+  if (!base::PathExists(result) && !base::CreateDirectory(result))

[Mark Mentovai, 2015/12/02 22:47:12]

The Crashpad handler doesn’t care if you start it without this existing. You can
take at least this blocking operation out of the startup path, although really
that just moves it into handler startup, which we do block on. But there’s no
reason to do it twice.

Dunno if there’s something else in Chrome that wants to see this directory, but
there shouldn’t be.

[scottmg, 2015/12/03 01:40:15]

Done.

+    return false;
+  *crash_dir = result;
+  return true;
+#else
   // By setting the BREAKPAD_DUMP_LOCATION environment variable, an alternate
   // location to write breakpad crash dumps can be set.
   scoped_ptr<base::Environment> env(base::Environment::Create());
   std::string alternate_crash_dump_location;
   if (env->GetVar("BREAKPAD_DUMP_LOCATION", &alternate_crash_dump_location)) {
     base::FilePath crash_dumps_dir_path =
         base::FilePath::FromUTF8Unsafe(alternate_crash_dump_location);
     PathService::Override(chrome::DIR_CRASH_DUMPS, crash_dumps_dir_path);
   }
 
   return PathService::Get(chrome::DIR_CRASH_DUMPS, crash_dir);
+#endif
 }
 
 size_t ChromeCrashReporterClient::RegisterCrashKeys() {
   return crash_keys::RegisterChromeCrashKeys();
 }
 
 bool ChromeCrashReporterClient::IsRunningUnattended() {
   scoped_ptr<base::Environment> env(base::Environment::Create());
   return env->HasVar(env_vars::kHeadless);
 }
@@ skipping 33 matching lines @@
 #endif
 
 bool ChromeCrashReporterClient::EnableBreakpadForProcess(
     const std::string& process_type) {
   return process_type == switches::kRendererProcess ||
          process_type == switches::kPluginProcess ||
          process_type == switches::kPpapiPluginProcess ||
          process_type == switches::kZygoteProcess ||
          process_type == switches::kGpuProcess;
 }