Skip to content

Commit

Permalink
[CrOS Cellular] Emit cellular setup result to correct histogram.
Browse files Browse the repository at this point in the history
Currently the setup flow result is being emitted to
Network.Cellular.{ESim|PSim}.CellularSetupResult when it should be
emitted to Network.Cellular.{ESim|PSim}.SetupFlowResult.

(cherry picked from commit 4b95ec2)

Bug: 1198673
Change-Id: Ibc30d9e3f20e61b2605018f6f4ea245817607261
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2824130
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Regan Hsu <hsuregan@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#872159}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2824931
Cr-Commit-Position: refs/branch-heads/4472@{#75}
Cr-Branched-From: 3d60439-refs/heads/master@{#870763}
  • Loading branch information
Regan Hsu authored and Chromium LUCI CQ committed Apr 14, 2021
1 parent 20a9d3d commit 900a413
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ cr.define('cellular_setup', function() {
};

/* #export */ const ESIM_SETUP_RESULT_METRIC_NAME =
'Network.Cellular.ESim.CellularSetupResult';
'Network.Cellular.ESim.SetupFlowResult';

/* #export */ const SUCCESSFUL_ESIM_SETUP_DURATION_METRIC_NAME =
'Network.Cellular.ESim.CellularSetup.Success.Duration';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ cr.define('cellularSetup', function() {
const MAX_START_ACTIVATION_ATTEMPTS = 3;

/* #export */ const PSIM_SETUP_RESULT_METRIC_NAME =
'Network.Cellular.PSim.CellularSetupResult';
'Network.Cellular.PSim.SetupFlowResult';

/* #export */ const SUCCESSFUL_PSIM_SETUP_DURATION_METRIC_NAME =
'Network.Cellular.PSim.CellularSetup.Success.Duration';
Expand Down

0 comments on commit 900a413

Please sign in to comment.