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

Issue 602060 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Not currently working on Chromium
Closed: Sep 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Feature

Blocking:
issue 488579
issue 609844



Sign in to add a comment

bluetooth: android: Record status codes of onServicesDiscovered

Project Member Reported by scheib@chromium.org, Apr 10 2016

Issue description

Record UMA of status codes returned to onServicesDiscovered in ChromeBluetoothDevice.java to enable reasoning about if we need more code to handle errors.

https://code.google.com/p/chromium/codesearch#chromium/src/device/bluetooth/android/java/src/org/chromium/device/bluetooth/ChromeBluetoothDevice.java&q=onServicesDiscovered%20file:BluetoothDevice.*java&sq=package:chromium&type=cs&l=175
 

Comment 1 by scheib@chromium.org, Apr 10 2016

Labels: M-52

Comment 2 by ortuno@chromium.org, May 13 2016

Blocking: 609844

Comment 3 by scheib@chromium.org, May 16 2016

Labels: -M-52
Owner: scheib@chromium.org
Status: Started (was: Available)
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 13 2016

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

commit d5980418932d12bfa1f6884473247961d1e4151b
Author: scheib <scheib@chromium.org>
Date: Sat Aug 13 05:22:16 2016

bluetooth: android: Record GATT status codes UMA.

Android's API is underdocumented, and to help understand what errors
are actually being encountered in the wild we are adding UMA to capture
the error code values for results of GATT operations.

BUG= 602060 

Review-Url: https://codereview.chromium.org/2230703002
Cr-Commit-Position: refs/heads/master@{#411878}

[modify] https://crrev.com/d5980418932d12bfa1f6884473247961d1e4151b/device/bluetooth/android/java/src/org/chromium/device/bluetooth/ChromeBluetoothDevice.java
[modify] https://crrev.com/d5980418932d12bfa1f6884473247961d1e4151b/tools/metrics/histograms/histograms.xml

Project Member

Comment 6 by bugdroid1@chromium.org, Sep 1 2016

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

commit bb5db37f3f445deed9f005ac0a4d9fdee2d5a60c
Author: scheib <scheib@chromium.org>
Date: Thu Sep 01 01:51:00 2016

bluetooth: android: Histograms updated to use histogram_suffixes

BUG= 602060 

Review-Url: https://codereview.chromium.org/2295063004
Cr-Commit-Position: refs/heads/master@{#415843}

[modify] https://crrev.com/bb5db37f3f445deed9f005ac0a4d9fdee2d5a60c/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Started)

Sign in to add a comment