Commit 9bafa21d authored by phillis's avatar phillis Committed by Commit Bot

DPWA: fix UMA metrics name for WebApp.Launcher.

Fix the code to use WebApp.Launcher.LaunchResult and
WebApp.Launcher.UpdateResult to match the ones in histograms.xml

Bug: none
Change-Id: Iece91aa20d3614ff2d711d7c765ec0837a63c23f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2341845
Commit-Queue: Phillis Tang <phillis@chromium.org>
Reviewed-by: default avatarDaniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#796087}
parent 33f9dbed
......@@ -20,7 +20,7 @@ void RecordPwaLauncherResult() {
result >= 0 &&
result <= static_cast<DWORD>(WebAppLauncherLaunchResult::kMaxValue)) {
base::UmaHistogramEnumeration(
"WebApp.Launcher.WebAppLauncherLaunchResult",
"WebApp.Launcher.LaunchResult",
static_cast<WebAppLauncherLaunchResult>(result));
}
......
......@@ -65,8 +65,8 @@ TEST_F(WebAppLauncherLogTest, RecordPwaLauncherResult) {
RecordPwaLauncherResult();
// |expected_value| should have been recorded to UMA.
histogram_tester.ExpectUniqueSample(
"WebApp.Launcher.WebAppLauncherLaunchResult", expected_value, 1);
histogram_tester.ExpectUniqueSample("WebApp.Launcher.LaunchResult",
expected_value, 1);
// PWALauncherResult should have been deleted from the registry.
EXPECT_FALSE(key_.HasValue(kPwaLauncherResult));
......
......@@ -50,8 +50,7 @@ void DeleteHardLinkOrCopyCallback(const base::FilePath& launcher_path,
}
void RecordWebAppLauncherUpdateResult(WebAppLauncherUpdateResult result) {
base::UmaHistogramEnumeration("WebApp.Launcher.WebAppLauncherUpdateResult",
result);
base::UmaHistogramEnumeration("WebApp.Launcher.UpdateResult", result);
}
// Replaces |launcher_path| with the one at |latest_version_path|. This is done
......
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