Rework GetStabilityFileForProcess for use by Crashpad integration

components/browser_watcher/stability_paths.h

Index: components/browser_watcher/stability_paths.h
diff --git a/components/browser_watcher/stability_paths.h b/components/browser_watcher/stability_paths.h
index d66718c6226d478c951fa38f6e65c96464a8a3ff..61824cda772406aaa3efa467a672d1f5885dfc12 100644
--- a/components/browser_watcher/stability_paths.h
+++ b/components/browser_watcher/stability_paths.h
@@ -7,16 +7,27 @@
 
 #include "base/files/file_path.h"
 #include "base/process/process.h"
+#include "build/build_config.h"
+
+#if defined(OS_WIN)
+#include <winsock2.h>
+#endif  // defined(OS_WIN)
 
 namespace browser_watcher {
 
 #if defined(OS_WIN)
 
-// Returns the the stability debugging directory.
+// Returns the stability debugging directory.
 base::FilePath GetStabilityDir(const base::FilePath& user_data_dir);
 
-// On success, |path| contains the path to the stability debugging information
-// file for |process|.
+// Returns the stability debugging path, which is based on pid and creation time
+// to ensure unicity in the face of pid recycling.

[Mark Mentovai, 2017/05/04 21:27:37]

uniqueness > uniquity > unicity, although they’re all fine.

[manzagop (departed), 2017/05/05 14:35:22]

Done.

+base::FilePath GetStabilityFileForProcess(base::ProcessId pid,
+                                          timeval creation_time,
+                                          const base::FilePath& user_data_dir);
+
+// On success, returns true and |path| contains the path to the stability file.
+// On failure, returns false.
 bool GetStabilityFileForProcess(const base::Process& process,
                                 const base::FilePath& user_data_dir,
                                 base::FilePath* path);