Add crash keys for the installer covering simple InstallerState fields.

chrome/installer/setup/installer_crash_reporter_client.cc

 // Copyright 2015 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/installer/setup/installer_crash_reporter_client.h"
 
 #include "base/debug/crash_logging.h"
 #include "base/environment.h"
 #include "base/file_version_info.h"
 #include "base/path_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/win/registry.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_version.h"
 #include "chrome/common/env_vars.h"
+#include "chrome/installer/setup/setup_constants.h"
 #include "chrome/installer/util/google_update_settings.h"
 #include "components/crash/core/common/crash_keys.h"
 
 InstallerCrashReporterClient::InstallerCrashReporterClient(
     bool is_per_user_install)
     : is_per_user_install_(is_per_user_install) {
 }
 
 InstallerCrashReporterClient::~InstallerCrashReporterClient() = default;
 
@@ skipping 93 matching lines @@
 bool InstallerCrashReporterClient::GetCrashDumpLocation(
     base::FilePath* crash_dir) {
   // TODO(grt): Is there a reason to support the BREAKPAD_DUMP_LOCATION
   // environment variable?
   return PathService::Get(chrome::DIR_CRASH_DUMPS, crash_dir);
 }
 
 size_t InstallerCrashReporterClient::RegisterCrashKeys() {
   const base::debug::CrashKey kKeys[] = {
     { crash_keys::kClientId, crash_keys::kSmallSize },
+    { installer::crash_keys::kCurrentVersion, crash_keys::kSmallSize },
+    { installer::crash_keys::kDistributionType, crash_keys::kSmallSize },
+    { installer::crash_keys::kOperation, crash_keys::kSmallSize },
+    { installer::crash_keys::kPackageType, crash_keys::kSmallSize },
+    { installer::crash_keys::kPrivilegeLevel, crash_keys::kSmallSize },
+
+    // This is a Windows registry key, which maxes out at 255 chars.
+    // (kMediumSize actually maxes out at 252 chars on Windows, but potentially
+    // truncating such a small amount is a fair tradeoff compared to using
+    // kLargeSize, which is wasteful.)
+    { installer::crash_keys::kStateKey, crash_keys::kMediumSize },
   };
 
   return base::debug::InitCrashKeys(&kKeys[0], arraysize(kKeys),
                                     crash_keys::kChunkMaxLength);
 }
 
 bool InstallerCrashReporterClient::IsRunningUnattended() {
   scoped_ptr<base::Environment> env(base::Environment::Create());
   return env->HasVar(env_vars::kHeadless);
 }
@@ skipping 37 matching lines @@
     }
   }
 
   return false;
 }
 
 bool InstallerCrashReporterClient::EnableBreakpadForProcess(
     const std::string& process_type) {
   return true;
 }