tree 486b25c4612380aee7340154a3efbe79b476e486
parent 33d1be1987c5532ee0253c01dd71541c8787cf74
author Nikhil Nayunigari <nikhilcn@google.com> 1714451051 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1714451051 +0000

[CrOS Hotspot] Fix inconsistencies in hotspot metrics

When hotspot is disabled due to user actions, we record metrics multiple
times: one from hotspot controller and other from state changes in
hotspot state handler. While hotspot controller records accurate reason
such as 'wifi_enabled' etc, we get a generic reason from hotspot state
handler such as 'user_initiated'.

With these changes, we are ignoring 'user_initiated' disable reason
being sent from hotspot_state_handler as all user initiated reasons
should be recorded from hotspot_controller.

(cherry picked from commit f9e5ac3e4b31f530f9d43d06c5b62834f773ed42)

Bug: b/333082297
Change-Id: I5b9cffc7f651c05b229e646b8802d13ded689de1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5486744
Commit-Queue: Nikhil Nayunigari <nikhilcn@google.com>
Reviewed-by: Jason Zhang <jiajunz@google.com>
Cr-Original-Commit-Position: refs/heads/main@{#1292552}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5499210
Cr-Commit-Position: refs/branch-heads/6422@{#495}
Cr-Branched-From: 9012208d0ce02e0cf0adb9b62558627c356f3278-refs/heads/main@{#1287751}