Closed Bug 1272671 Opened 10 years ago Closed 9 years ago

Move ProcessedStack and ThreadHangStats implementation details into separate modules

Categories

(Toolkit :: Telemetry, defect, P3)

Product:
Toolkit
Component:
Telemetry
Type:
defect
Points:
2

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1335461
Tracking Flags:
Tracking Status
firefox49 --- affected

People

(Reporter: gfritzsche, Unassigned)

References

Details

(Whiteboard: [measurement:client])

They are declared in separate headers but (at least partially) implemented in Telemetry.cpp. From bug 1258183, comment 16: @@ +2475,5 @@ > +//////////////////////////////////////////////////////////////////////// > +//////////////////////////////////////////////////////////////////////// > +// > +// EXTERNALLY VISIBLE FUNCTIONS in mozilla::Telemetry:: > +// These are NOT listed in Telemetry.h So, if i got this right, the remaining ones that are public and not listed in Telemetry.h are: ProcessedStack.h: ProcessedStack ThreadHangStats.h: TimeHistogram, HangStack, HangHistogram, ThreadHangStats I should fix that in a follow-up bug and maybe just properly move them out into separate modules.
Points: --- → 2
Priority: P2 → P3
See Also: → 1335461
We will fix this as part of bug 1335461.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.