Stability instrumentation: metrics for collection on crash

components/browser_watcher/stability_paths.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_STABILITY_PATHS_H_
 #define COMPONENTS_BROWSER_WATCHER_STABILITY_PATHS_H_
 
 #include "base/files/file_path.h"
 #include "base/process/process.h"
 #include "build/build_config.h"
@@ skipping 17 matching lines @@
 
 // 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);
 
 // 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)
 
 }  // namespace browser_watcher
 
 #endif  // COMPONENTS_BROWSER_WATCHER_STABILITY_PATHS_H_