New issue
Advanced search Search tips

Issue 910150 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 10
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Split up NO_RESPONSE bucket

Project Member Reported by hansberry@google.com, Nov 29

Issue description

(From b/119840026)

Right not, this bucket is very common, but it doesn't disambiguate
if it's due to the phone never being connected to, or the phone
receiving a request but never responding. The bucket should be split into those two, in order to determine which error case
needs to be worked on.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 7

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

commit 38192c50331a28876756a685473f2e29b2f14c2e
Author: Ryan Hansberry <hansberry@chromium.org>
Date: Fri Dec 07 23:43:09 2018

Tether: Split up metrics bucket for 'no response' from host.

Right now, this bucket is very common, but it doesn't disambiguate
if it's due to the phone never being connected to, or the phone
receiving a request but never responding. This CL splits the
bucket into those two, in order to determine which error case
needs to be worked on.

Bug:  910150 
Change-Id: I552f0810348cbf8efce5b6db44341511962de7f0
Reviewed-on: https://chromium-review.googlesource.com/c/1354691
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Ryan Hansberry <hansberry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#614868}
[modify] https://crrev.com/38192c50331a28876756a685473f2e29b2f14c2e/chromeos/components/tether/connect_tethering_operation.cc
[modify] https://crrev.com/38192c50331a28876756a685473f2e29b2f14c2e/chromeos/components/tether/connect_tethering_operation.h
[modify] https://crrev.com/38192c50331a28876756a685473f2e29b2f14c2e/chromeos/components/tether/host_connection_metrics_logger.cc
[modify] https://crrev.com/38192c50331a28876756a685473f2e29b2f14c2e/chromeos/components/tether/host_connection_metrics_logger.h
[modify] https://crrev.com/38192c50331a28876756a685473f2e29b2f14c2e/chromeos/components/tether/tether_connector_impl.cc
[modify] https://crrev.com/38192c50331a28876756a685473f2e29b2f14c2e/chromeos/components/tether/tether_connector_impl.h
[modify] https://crrev.com/38192c50331a28876756a685473f2e29b2f14c2e/tools/metrics/histograms/enums.xml

Status: Fixed (was: Started)

Sign in to add a comment