Stability instrumentation: metrics for collection on crash

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 07216f1f75cb4a241b2b06d67bf8ccd9fc7cab00..1d8cde265718d9889520ff8bcc39da6ccd603834 100644
--- a/components/browser_watcher/stability_paths.h
+++ b/components/browser_watcher/stability_paths.h
@@ -35,8 +35,11 @@ bool GetStabilityFileForProcess(const base::Process& process,
 // Returns a pattern that matches file names returned by GetFileForProcess.
 base::FilePath::StringType GetStabilityFilePattern();
 
-// Marks the stability file for deletion.
-void MarkStabilityFileForDeletion(const base::FilePath& user_data_dir);
+// Marks the current process's stability file for deletion.
+void MarkOwnStabilityFileDeleted(const base::FilePath& user_data_dir);
+
+// Marks a stability file for deletion on crash. Meant for the crashpad handler.

[Sigurður Ásgeirsson, 2017/06/01 12:50:45]

nit: This description seems pretty ambiguous and redundant to the name of the
function.

Maybe you want to mention that some of these functions log specific kinds of
events?

[manzagop (departed), 2017/06/01 13:45:12]

+1 Hopefully better now. Done.

+void MarkStabilityFileDeletedOnCrash(const base::FilePath& file_path);
 
 #endif  // defined(OS_WIN)