Add crash keys for the installer covering simple InstallerState fields.

chrome/installer/setup/setup_constants.cc

Index: chrome/installer/setup/setup_constants.cc
diff --git a/chrome/installer/setup/setup_constants.cc b/chrome/installer/setup/setup_constants.cc
index 8d705faad8100a49bf1f8cea4acc9c31b667ed2c..1cd7caa543dacb88d1b53b7d7feadc2671c8968a 100644
--- a/chrome/installer/setup/setup_constants.cc
+++ b/chrome/installer/setup/setup_constants.cc
@@ -35,4 +35,15 @@ const char kSetDisplayVersionValue[] = "set-display-version-value";
 
 }  // namespace switches
 
+namespace crash_keys {
+
+const char kCurrentVersion[] = "current-version";
+const char kDistributionType[] = "dist-type";
+const char kOperation[] = "operation";
+const char kPackageType[] = "package-type";

[grt (UTC plus 2), 2015/11/25 16:44:09]

since this is keyed off the --multi-install command line switch, i think
"multi-install" with values of ("false", "true") will be easier to understand on
the backend.

[Joe Mason, 2015/11/25 17:20:48]

Shouldn't we plan for expansion such as adding further package types? (I'm
following the example of the comment saying InstallerState::is_multi_install is
deprecated, so trying to map to the enum values instead of a bool.)

[grt (UTC plus 2), 2015/11/25 17:44:01]

Yeah, about that... :-) We're working on removing the concept of package types
and multi-install. There will never be more than single and multi, and multi
will go away leaving only single. When that happens we'll delete this crash key
altogether.

+const char kPrivilegeLevel[] = "priv-level";

[grt (UTC plus 2), 2015/11/25 16:44:09]

likewise, i think "system-level" with ("false", "true") is better here.

+const char kStateKey[] = "state-key";
+
+}  // namespace crash_keys
+
 }  // namespace installer