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

Issue 675375 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

Upstreaming followup: add PhysicalWeb.* to tools/metrics/histograms/histograms.xml

Project Member Reported by sdefresne@chromium.org, Dec 17 2016

Issue description

Presubmit warning when upstreaming ios/chrome/today_extension code:

Some UMA_HISTOGRAM lines have been modified and the associated histogram name has no match in either tools/metrics/histograms/histograms.xml or the modifications of it:
   [ios/chrome/today_extension/today_view_controller.mm:885] PhysicalWeb.TotalBeaconsDetected \
   [ios/chrome/today_extension/today_view_controller.mm:929] PhysicalWeb.InitialState

 
Status: Started (was: Assigned)
Cc: noyau@chromium.org
noyau: those are followup to upstreaming work.

Some of them can already be addressed now (UMA histograms and actions.xml). If you want, please feel free to split in smaller issues and assign them to owners.
Components: Internals>Metrics

Comment 4 by holte@chromium.org, Aug 22 2017

Components: -Internals>Metrics
Labels: Hotlist-Metrics
Components: Internals>PhysicalWeb
Status: WontFix (was: Started)

Sign in to add a comment