Bound the impact of system instability on chrome instability.

components/browser_watcher/system_session_analyzer_win.h

+// Copyright 2017 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.
+
+#ifndef COMPONENTS_BROWSER_WATCHER_SYSTEM_SESSION_ANALYZER_WIN_H_
+#define COMPONENTS_BROWSER_WATCHER_SYSTEM_SESSION_ANALYZER_WIN_H_
+
+#include <map>
+#include <memory>
+#include <vector>
+
+#include "base/time/time.h"
+
+namespace browser_watcher {
+
+// Queries system logs for system session events.
+class SystemSessionEventFetcher {
+ public:
+  // Minimal information about a log event.
+  struct EventInfo {
+    uint16_t event_id;
+    base::Time event_time;
+  };
+
+  // Create a SystemSessionEventFetcher that will query for events pertaining to
+  // the last session_cnt sessions.
+  explicit SystemSessionEventFetcher(size_t session_cnt);
+
+  // Query for events pertaining to the last session_cnt sessions. On success,
+  // returns true and event_infos contains events ordered from newest to oldest.
+  // Returns false otherwise.
+  virtual bool FetchEvents(std::vector<EventInfo>* event_infos) const;
+
+ private:
+  size_t session_cnt_;
+
+  DISALLOW_COPY_AND_ASSIGN(SystemSessionEventFetcher);
+};
+
+// Analyzes system session events for unclean sessions.
+class SystemSessionAnalyzer {
+ public:
+  enum Status {
+    FAILED = 0,
+    CLEAN = 1,
+    UNCLEAN = 2,
+    OUTSIDE_RANGE = 3,
+  };
+
+  explicit SystemSessionAnalyzer(std::unique_ptr<SystemSessionEventFetcher>);
+  virtual ~SystemSessionAnalyzer() = default;
+
+  virtual Status IsSessionUnclean(base::Time timestamp);

[Will Harris, 2017/02/27 22:40:11]

comment?

[manzagop (departed), 2017/02/28 14:51:51]

Done.

+
+ private:
+  void Initialize();
+
+  std::unique_ptr<SystemSessionEventFetcher> event_fetcher_;
+
+  bool initialized_ = false;
+  bool init_success_ = false;
+
+  // Information about unclean sessions: start time to duration until the next
+  // session start, ie *not* session duration. Note: it's easier to get the
+  // delta to the next session start, and changes nothing wrt classiying
+  // events that occur during sessions assuming query timestamps fall within
+  // system sessions.
+  std::map<base::Time, base::TimeDelta> unclean_sessions_;
+
+  // Timestamp of the oldest event.
+  base::Time coverage_start_;
+
+  DISALLOW_COPY_AND_ASSIGN(SystemSessionAnalyzer);
+};
+
+}  // namespace browser_watcher
+
+#endif  // COMPONENTS_BROWSER_WATCHER_SYSTEM_SESSION_ANALYZER_WIN_H_