Record keep alive registry's state in stability instrumentation

chrome/browser/lifetime/keep_alive_registry.cc

Index: chrome/browser/lifetime/keep_alive_registry.cc
diff --git a/chrome/browser/lifetime/keep_alive_registry.cc b/chrome/browser/lifetime/keep_alive_registry.cc
index f61ef82706df2cd271f97e0e78fa3266f35d7a13..64ead1c72d159e9921c9b4fc86b5e9e501b75c91 100644
--- a/chrome/browser/lifetime/keep_alive_registry.cc
+++ b/chrome/browser/lifetime/keep_alive_registry.cc
@@ -4,11 +4,17 @@
 
 #include "chrome/browser/lifetime/keep_alive_registry.h"
 
+#include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/lifetime/keep_alive_state_observer.h"
 #include "chrome/browser/lifetime/keep_alive_types.h"
 
+#if defined(OS_WIN)

[bcwhite, 2017/05/08 15:37:59]

Why do this only on Windows?

[manzagop (departed), 2017/05/08 16:12:59]

I recently re-jigged the build targets for reuse by the crashing scenario and
felt it was simpler to make all build targets windows only for now, instead of
dealing with the complexity of multiple platforms (figuring what's is_win and
what's not among the dependencies). I'm thinking we can change this once we
tackle the other platforms.

+#include "components/browser_watcher/stability_data_names.h"
+#include "components/browser_watcher/stability_debugging.h"
+#endif  // defined(OS_WIN)
+
 ////////////////////////////////////////////////////////////////////////////////
 // Public methods
 
@@ -136,6 +142,10 @@ void KeepAliveRegistry::Unregister(KeepAliveOrigin origin,
 void KeepAliveRegistry::OnKeepAliveStateChanged(bool new_keeping_alive) {
   DVLOG(1) << "Notifying KeepAliveStateObservers: KeepingAlive changed to: "
            << new_keeping_alive;
+#if defined(OS_WIN)
+  browser_watcher::SetStabilityDataBool(browser_watcher::kStabilityKeepAlive,
+                                        new_keeping_alive);
+#endif  // defined(OS_WIN)
   for (KeepAliveStateObserver& observer : observers_)
     observer.OnKeepAliveStateChanged(new_keeping_alive);
 }
@@ -143,6 +153,10 @@ void KeepAliveRegistry::OnKeepAliveStateChanged(bool new_keeping_alive) {
 void KeepAliveRegistry::OnRestartAllowedChanged(bool new_restart_allowed) {
   DVLOG(1) << "Notifying KeepAliveStateObservers: Restart changed to: "
            << new_restart_allowed;
+#if defined(OS_WIN)
+  browser_watcher::SetStabilityDataBool(
+      browser_watcher::kStabilityRestartAllowed, new_restart_allowed);
+#endif  // defined(OS_WIN)
   for (KeepAliveStateObserver& observer : observers_)
     observer.OnKeepAliveRestartStateChanged(new_restart_allowed);
 }