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/logging.h"
 #include "base/path_service.h"
 #include "base/strings/string_split.h"
 #include "base/strings/utf_string_conversions.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 chrome's PathService entries (for DIR_CRASH_DUMPS) being available on

[Mark Mentovai, 2015/12/03 15:11:32]

Lose “the”.

[scottmg, 2015/12/03 18:14:51]

Done.

+  // Windows. See http://crbug.com/564398.
+  base::FilePath result;
+  if (!PathService::Get(base::DIR_LOCAL_APP_DATA, &result))

[Mark Mentovai, 2015/12/03 15:11:32]

Following up on the --user-data-dir discussion:

The DIR_CRASH_DUMPS getter comes with this comment:
which becomes moot if you honor --user-data-dir here.

You can leave it as-is for now and we can save the change for a follow-up fix,
but now that we’ve got a distinct but equivalent way to get DIR_CRASH_DUMPS, the
rationale for not respecting --user-data-dir is shaky.

My main concern is that Chrome treats the consent bit as per-user-data-dir, but
if multiple user-data-dirs share the same Crashpad settings database, things can
easily get out of sync.

It looks like we have this problem on Mac now too, so maybe it’s best to save it
for a follow-up fix after all.

[scottmg, 2015/12/03 18:14:51]

Yeah, that's a good point. I filed crbug.com/565466, and tagged this location
and DIR_CRASH_DUMPS with the bug, but didn't change anything for now.

I don't think anything outside of //components/crash actually needs
DIR_CRASH_DUMPS when using Crashpad, so for Mac and Win we could just make the
canonical location GetCrashDumpLocation() and not support DIR_CRASH_DUMPS at
all.

+    return false;
+  BrowserDistribution* dist = BrowserDistribution::GetDistribution();
+  result = result.Append(dist->GetInstallSubDir());
+  result = result.Append(chrome::kUserDataDirname);
+  result = result.Append(FILE_PATH_LITERAL("Crashpad"));
+  *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;
 }