Commit cc109e2c authored by Sergei Datsenko's avatar Sergei Datsenko Committed by Commit Bot

Change bucketing of drivefs metrics

Apparently our times saturate the default bucketing and we want wider.

BUG=None

Change-Id: I03d5c0ef99107c4025e5d8a67e4d2e85133dcb3b
Reviewed-on: https://chromium-review.googlesource.com/c/1300355
Commit-Queue: Sergei Datsenko <dats@chromium.org>
Reviewed-by: default avatarSam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602985}
parent d8672845
...@@ -346,11 +346,11 @@ void UmaEmitMountOutcome(DriveMountStatus status, ...@@ -346,11 +346,11 @@ void UmaEmitMountOutcome(DriveMountStatus status,
const base::TimeTicks& time_started) { const base::TimeTicks& time_started) {
UMA_HISTOGRAM_ENUMERATION("DriveCommon.Lifecycle.Mount", status); UMA_HISTOGRAM_ENUMERATION("DriveCommon.Lifecycle.Mount", status);
if (status == DriveMountStatus::kSuccess) { if (status == DriveMountStatus::kSuccess) {
UMA_HISTOGRAM_TIMES("DriveCommon.Lifecycle.MountTime.SuccessTime", UMA_HISTOGRAM_MEDIUM_TIMES("DriveCommon.Lifecycle.MountTime.SuccessTime",
base::TimeTicks::Now() - time_started); base::TimeTicks::Now() - time_started);
} else { } else {
UMA_HISTOGRAM_TIMES("DriveCommon.Lifecycle.MountTime.FailTime", UMA_HISTOGRAM_MEDIUM_TIMES("DriveCommon.Lifecycle.MountTime.FailTime",
base::TimeTicks::Now() - time_started); base::TimeTicks::Now() - time_started);
} }
} }
...@@ -359,8 +359,8 @@ void UmaEmitUnmountOutcome(DriveMountStatus status) { ...@@ -359,8 +359,8 @@ void UmaEmitUnmountOutcome(DriveMountStatus status) {
} }
void UmaEmitFirstLaunch(const base::TimeTicks& time_started) { void UmaEmitFirstLaunch(const base::TimeTicks& time_started) {
UMA_HISTOGRAM_TIMES("DriveCommon.Lifecycle.FirstLaunchTime", UMA_HISTOGRAM_MEDIUM_TIMES("DriveCommon.Lifecycle.FirstLaunchTime",
base::TimeTicks::Now() - time_started); base::TimeTicks::Now() - time_started);
} }
} // namespace } // namespace
......
...@@ -862,11 +862,17 @@ void UmaEmitSearchOutcome( ...@@ -862,11 +862,17 @@ void UmaEmitSearchOutcome(
infix = "OfflineSearchTime"; infix = "OfflineSearchTime";
break; break;
} }
base::UmaHistogramTimes( if (remote) {
base::StrCat( base::UmaHistogramMediumTimes(
{remote ? "DriveCommon.RemoteSearch." : "DriveCommon.LocalSearch.", base::StrCat({"DriveCommon.RemoteSearch.", infix,
infix, success ? ".SuccessTime" : ".FailTime"}), success ? ".SuccessTime" : ".FailTime"}),
base::TimeTicks::Now() - time_started); base::TimeTicks::Now() - time_started);
} else {
base::UmaHistogramMediumTimes(
base::StrCat({"DriveCommon.LocalSearch.", infix,
success ? ".SuccessTime" : ".FailTime"}),
base::TimeTicks::Now() - time_started);
}
} }
} // namespace } // namespace
......
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