Record MetricsService's execution phase to the stability file

components/browser_watcher/stability_debugging.cc

 // 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_debugging_win.h"
+#include "components/browser_watcher/stability_debugging.h"
 
 #include <string>
 
+#include "base/debug/activity_tracker.h"
 #include "base/feature_list.h"
 #include "base/files/file.h"
 #include "base/metrics/persistent_memory_allocator.h"
 #include "base/path_service.h"
 #include "base/process/process.h"
 #include "base/strings/stringprintf.h"
 #include "base/time/time.h"
 #include "components/browser_watcher/features.h"
 
 namespace browser_watcher {
 
 namespace {
 
+#if defined(OS_WIN)
 bool GetCreationTime(const base::Process& process, base::Time* time) {

[bcwhite, 2016/12/05 17:20:57]

Are these really windows-only?

[manzagop (departed), 2016/12/06 20:35:08]

The current implementation is, because it relies on ::GetProcessTimes.

   DCHECK(time);
 
   FILETIME creation_time = {};
   FILETIME ignore1 = {};
   FILETIME ignore2 = {};
   FILETIME ignore3 = {};
   if (!::GetProcessTimes(process.Handle(), &creation_time, &ignore1, &ignore2,
                          &ignore3)) {
     return false;
   }
   *time = base::Time::FromFileTime(creation_time);
   return true;
 }
+#endif  // defined(OS_WIN)
 
 }  // namespace
 
+#if defined(OS_WIN)
+
 base::FilePath GetStabilityDir(const base::FilePath& user_data_dir) {
   return user_data_dir.AppendASCII("Stability");
 }
 
 bool GetStabilityFileForProcess(const base::Process& process,
                                 const base::FilePath& user_data_dir,
                                 base::FilePath* file_path) {
   DCHECK(file_path);
   base::FilePath stability_dir = GetStabilityDir(user_data_dir);
 
@@ skipping 28 matching lines @@
     return;
   }
 
   // Open (with delete) and then immediately close the file by going out of
   // scope. This should cause the stability debugging file to be deleted prior
   // to the next execution.
   base::File file(stability_file, base::File::FLAG_OPEN |
                                       base::File::FLAG_READ |
                                       base::File::FLAG_DELETE_ON_CLOSE);
 }
+#endif  // defined(OS_WIN)
+
+void SetStabilityDataInt(base::StringPiece name, int64_t value) {
+  base::debug::GlobalActivityTracker* global_tracker =
+      base::debug::GlobalActivityTracker::Get();
+  if (!global_tracker)
+    return;  // Activity tracking isn't enabled.
+
+  global_tracker->user_data().SetInt(name, value);
+}
 
 }  // namespace browser_watcher