Commit 733204c1 authored by Ben Wells's avatar Ben Wells Committed by Commit Bot

Record metric for time from device setup to Crostini installation.

We are interested in knowing how long it takes users from first setting
up a device to setting up Crostini.

Bug: 703927
Change-Id: I68d24b4f6f408b3f5dce853dfaaa40c10e1d72be
Reviewed-on: https://chromium-review.googlesource.com/1082068Reviewed-by: default avatarNicholas Verne <nverne@chromium.org>
Reviewed-by: default avatarIlya Sherman <isherman@chromium.org>
Commit-Queue: Ben Wells <benwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#567062}
parent 543be0a1
......@@ -14,6 +14,7 @@
#include "chrome/browser/chromeos/crostini/crostini_manager.h"
#include "chrome/browser/chromeos/crostini/crostini_pref_names.h"
#include "chrome/browser/chromeos/crostini/crostini_util.h"
#include "chrome/browser/chromeos/login/startup_utils.h"
#include "chrome/browser/chromeos/profiles/profile_helper.h"
#include "chrome/browser/ui/browser_dialogs.h"
#include "chrome/browser/ui/views/harmony/chrome_layout_provider.h"
......@@ -43,6 +44,19 @@ constexpr int kDownloadSizeInBytes = 300 * 1024 * 1024;
constexpr char kCrostiniSetupResultHistogram[] = "Crostini.SetupResult";
constexpr char kCrostiniSetupSourceHistogram[] = "Crostini.SetupSource";
constexpr char kCrostiniTimeFromDeviceSetupToInstall[] =
"Crostini.TimeFromDeviceSetupToInstall";
void RecordTimeFromDeviceSetupToInstallMetric() {
base::TimeDelta time_from_device_setup(
chromeos::StartupUtils::GetTimeSinceOobeFlagFileCreation());
if (time_from_device_setup.is_zero())
return;
base::UmaHistogramCustomTimes(
kCrostiniTimeFromDeviceSetupToInstall, time_from_device_setup,
base::TimeDelta::FromMinutes(1), base::TimeDelta::FromDays(365), 50);
}
} // namespace
......@@ -280,6 +294,7 @@ void CrostiniInstallerView::ShowLoginShell() {
StepProgress();
RecordSetupResultHistogram(SetupResult::kSuccess);
RecordTimeFromDeviceSetupToInstallMetric();
GetWidget()->Close();
}
......
......@@ -14261,6 +14261,14 @@ uploading your change for review.
</summary>
</histogram>
<histogram name="Crostini.TimeFromDeviceSetupToInstall" units="ms">
<owner>benwells@chromium.org</owner>
<owner>tbuckley@chromium.org</owner>
<summary>
The time from a user setting up their device, to the user enabling Crostini.
</summary>
</histogram>
<histogram name="Crostini.UninstallResult" enum="CrostiniUninstallResult">
<owner>benwells@chromium.org</owner>
<owner>tbuckley@chromium.org</owner>
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment