Investigate if flag histogram types are broken in Fennec

NEW
Unassigned

Status

()

Firefox for Android
General
3 years ago
3 years ago

People

(Reporter: mhaigh, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

3 years ago
Bug 1159753 introduced a flag histogram type which subsequently caused test failures.  We don't use flag types anywhere else in the app and changing the flag to a boolean seems to fix the issue, so it's possible that flag types are broken.  

The failures point to build/src/ipc/chromium/src/base/histogram.cc, line 1020

Investigate and fix.
(Reporter)

Updated

3 years ago
Assignee: mhaigh → nobody

Comment 1

3 years ago
jchen, do you know what could be going wrong here? I would like us to be able to use flag histograms from Java, but this problem prevents us from doing that.
Flags: needinfo?(nchen)
AFAIU flag histogram only expects "1" as the histogram value, to turn the flag on. To keep the value at 0, you would simply skip the add telemetry call.
Flags: needinfo?(nchen)
You need to log in before you can comment on or make changes to this bug.