Wire in postmortem report collection

components/browser_watcher/stability_debugging_win.h

+// Copyright (c) 2016 The Chromium Authors. All rights reserved.

[Alexei Svitkine (slow), 2016/09/23 14:13:53]

Nit: No (c) for new files

[manzagop (departed), 2016/09/23 21:57:01]

Done. Here and elsewhere.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_BROWSER_WATCHER_STABILITY_DEBUGGING_WIN_H_
+#define COMPONENTS_BROWSER_WATCHER_STABILITY_DEBUGGING_WIN_H_
+
+#include "base/files/file_path.h"
+#include "base/gtest_prod_util.h"
+#include "base/process/process.h"
+
+namespace browser_watcher {
+
+class StabilityDebugging {

[Alexei Svitkine (slow), 2016/09/23 14:13:53]

Is the extra class wrapper really necessary for these? I'd just making them
free-standing functions.

Style guide says "Prefer grouping functions with a namespace instead of using a
class as if it were a namespace. " - and seems like you already have a
namespace.

[manzagop (departed), 2016/09/23 21:57:01]

I added "Stability" to the function names. Done.

+ public:
+  // Returns the the stability debugging directory.
+  static base::FilePath GetDir(const base::FilePath& user_data_dir);
+
+  // On success, |path| contains the path to the stability debugging information
+  // file for |process|.
+  static bool GetFileForProcess(const base::Process& process,
+                                const base::FilePath& user_data_dir,
+                                base::FilePath* path);
+
+  // Returns a pattern that matches file names returned by GetFileForProcess.
+  static base::FilePath::StringType GetFilePattern();
+};
+
+}  // namespace browser_watcher
+
+#endif  // COMPONENTS_BROWSER_WATCHER_STABILITY_DEBUGGING_WIN_H_