| commit | ea27a2cd67f3d3fdeb5b27abb586927cb9d39706 | [log] [tgz] |
|---|---|---|
| author | Greg Thompson <[email protected]> | Tue Jun 25 19:22:22 2019 |
| committer | Commit Bot <[email protected]> | Tue Jun 25 19:22:22 2019 |
| tree | 932ea948b6a550e725f79880e5e608d596262b5e | |
| parent | 6e2a83e07b019baccc45ca6f793ba62ed5c6a16a [diff] [blame] |
Deprecate SettingsAppMonitor.InitializationResult. All signs indicate that it is exceptionally rare for initialization to fail. BUG=975750 [email protected],[email protected] Change-Id: I5cee019454c882c9702d69d3c4877d7cda97d5a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1670889 Commit-Queue: Jesse Doherty <[email protected]> Reviewed-by: Jesse Doherty <[email protected]> Reviewed-by: Patrick Monette <[email protected]> Auto-Submit: Greg Thompson <[email protected]> Cr-Commit-Position: refs/heads/master@{#672191}
diff --git a/chrome/browser/shell_integration_win.cc b/chrome/browser/shell_integration_win.cc index 74b5b38..4ee45c5 100644 --- a/chrome/browser/shell_integration_win.cc +++ b/chrome/browser/shell_integration_win.cc
@@ -249,8 +249,7 @@ private: // win::SettingsAppMonitor::Delegate: void OnInitialized(HRESULT result) override { - UMA_HISTOGRAM_BOOLEAN("SettingsAppMonitor.InitializationResult", - SUCCEEDED(result)); + // UMA indicates that this succeeds > 99.98% of the time. if (SUCCEEDED(result)) { base::RecordAction( base::UserMetricsAction("SettingsAppMonitor.Initialized"));