New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 902868 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 14
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Report to MultiDevice.Setup.HostStatus each time it changes

Project Member Reported by hansberry@chromium.org, Nov 7

Issue description

Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 14

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bdd7f48bfb3d65d5097c1aa4383bcc2ef2615f74

commit bdd7f48bfb3d65d5097c1aa4383bcc2ef2615f74
Author: Regan Hsu <hsuregan@chromium.org>
Date: Wed Nov 14 00:21:03 2018

[CrOS MultiDevice] Modify MultiDevice.Setup.HostStatus metrics to
account for every time status changes its value

Change location of host status collection during MultiDevice Setup.
The metric was logged only on start-up.  This change intends for
HostStatus to log on start up and every time status changes its value.

Bug:  902868 
Change-Id: I30de454dee61ff8bf8f5cd8b6ab071fc820abd35
Reviewed-on: https://chromium-review.googlesource.com/c/1327415
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Ryan Hansberry <hansberry@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Regan Hsu <hsuregan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#607827}
[modify] https://crrev.com/bdd7f48bfb3d65d5097c1aa4383bcc2ef2615f74/chromeos/services/multidevice_setup/host_status_provider_impl.cc
[modify] https://crrev.com/bdd7f48bfb3d65d5097c1aa4383bcc2ef2615f74/chromeos/services/multidevice_setup/public/mojom/multidevice_setup.mojom
[modify] https://crrev.com/bdd7f48bfb3d65d5097c1aa4383bcc2ef2615f74/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Started)

Sign in to add a comment