Delete the stability file on clean exit.

chrome/browser/metrics/chrome_metrics_service_client.cc

 // Copyright 2014 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.
 
 #include "chrome/browser/metrics/chrome_metrics_service_client.h"
 
 #include <stddef.h>
 
 #include <utility>
 #include <vector>
@@ skipping 87 matching lines @@
 #endif
 
 #if defined(OS_WIN)
 #include <windows.h>
 
 #include "chrome/browser/metrics/antivirus_metrics_provider_win.h"
 #include "chrome/browser/metrics/google_update_metrics_provider_win.h"
 #include "chrome/common/metrics_constants_util_win.h"
 #include "chrome/install_static/install_util.h"
 #include "chrome/installer/util/browser_distribution.h"
+#include "components/browser_watcher/stability_debugging_win.h"
 #include "components/browser_watcher/watcher_metrics_provider_win.h"
 #endif
 
 #if defined(OS_WIN) || defined(OS_MACOSX)
 #include "third_party/crashpad/crashpad/client/crashpad_info.h"
 #endif
 
 #if !defined(OS_CHROMEOS)
 #include "chrome/browser/signin/chrome_signin_status_metrics_provider_delegate.h"
 #include "components/signin/core/browser/signin_status_metrics_provider.h"
@@ skipping 132 matching lines @@
   DCHECK_NE(nullptr, channel_name);
 
   wchar_t exe_file[MAX_PATH] = {};
   CHECK(::GetModuleFileName(nullptr, exe_file, arraysize(exe_file)));
 
   base::string16 unused_special_build;
   install_static::GetExecutableVersionDetails(
       exe_file, product_name, version_number, &unused_special_build,
       channel_name);
 }
+

[Alexei Svitkine (slow), 2016/11/22 22:22:19]

Nit: Remove from the diff

[manzagop (departed), 2016/11/23 16:35:49]

Done.

 #endif  // OS_WIN
 
 }  // namespace
 
 const char ChromeMetricsServiceClient::kBrowserMetricsName[] = "BrowserMetrics";
 
 ChromeMetricsServiceClient::ChromeMetricsServiceClient(
     metrics::MetricsStateManager* state_manager)
     : metrics_state_manager_(state_manager),
 #if defined(OS_CHROMEOS)
@@ skipping 117 matching lines @@
 #endif  // OS_WIN || OS_MACOSX
 }
 
 void ChromeMetricsServiceClient::OnLogUploadComplete() {
   // Collect time ticks stats after each UMA upload.
 #if defined(OS_WIN)
   chrome::CollectTimeTicksStats();
 #endif
 }
 
+void ChromeMetricsServiceClient::OnLogCleanShutdown() {
+#if defined(OS_WIN)
+  base::FilePath user_data_dir;
+  if (!base::PathService::Get(chrome::DIR_USER_DATA, &user_data_dir)) {
+    // TODO(manzagop): add a metric.
+    return;
+  }
+  browser_watcher::MarkStabilityFileForDeletion(user_data_dir);
+#endif  // OS_WIN
+}
+
 void ChromeMetricsServiceClient::InitializeSystemProfileMetrics(
     const base::Closure& done_callback) {
   DCHECK(initialize_task_queue_.empty());
 
   // Each provider's initializer takes its own "done_callback" to enable
   // asynchronously chaining. We bind |next_task| to this callback, so
   // OnInitNextTask() gets called after each initializer, and we can go through
   // successive tasks in |initialize_task_queue_|. Note that |next_task| is
   // copied by value when passed into base::Bind().
   base::Closure next_task =
@@ skipping 472 matching lines @@
   }
 }
 
 void ChromeMetricsServiceClient::OnURLOpenedFromOmnibox(OmniboxLog* log) {
   metrics_service_->OnApplicationNotIdle();
 }
 
 bool ChromeMetricsServiceClient::IsUMACellularUploadLogicEnabled() {
   return metrics::IsCellularLogicEnabled();
 }