Record executable version details to stability file

components/browser_watcher/stability_data_names.cc

Index: components/browser_watcher/stability_data_names.cc
diff --git a/components/browser_watcher/stability_data_names.cc b/components/browser_watcher/stability_data_names.cc
new file mode 100644
index 0000000000000000000000000000000000000000..b227065d520f4dc8b2e9111cd562dcbdd86a8a02
--- /dev/null
+++ b/components/browser_watcher/stability_data_names.cc
@@ -0,0 +1,15 @@
+// Copyright 2016 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/browser_watcher/stability_data_names.h"
+
+namespace browser_watcher {
+
+const char kStabilityChannel[] = "channel";

[bcwhite, 2016/12/06 19:28:43]

Since these are global, is it worth having the strings indicate such?  e.g.
Chrome-Channel
Chrome-Platform
etc.

Or similar.  Just wondering if there might end up being, for example, a
"version" of something else that is also global.

[manzagop (departed), 2016/12/06 21:39:03]

I think I prefer keeping these short and we can qualify more the more specific
ones. Sg?

[bcwhite, 2016/12/06 22:37:32]

Your call.

+const char kStabilityPlatform[] = "platform";
+const char kStabilityProduct[] = "product";
+const char kStabilitySpecialBuild[] = "special-build";
+const char kStabilityVersion[] = "version";
+
+}  // namespace browser_watcher