Do not include StartupTimline.h in Telemetry.h

RESOLVED FIXED in Firefox 68

Status

()

enhancement
RESOLVED FIXED
3 months ago
3 months ago

People

(Reporter: gsvelto, Assigned: gsvelto)

Tracking

(Blocks 1 bug)

unspecified
mozilla68
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox68 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

3 months ago

While working on something unrelated I noticed that touching StartupTimeline.h would cause half of the tree to be rebuilt. It turns out that it's because it's been included by Telemetry.h. Since it doesn't need it (it's used in Telemetry.cpp which includes it already) we can remove it.

Assignee

Updated

3 months ago
Assignee: nobody → gsvelto
Status: NEW → ASSIGNED

Comment 2

3 months ago
Pushed by gsvelto@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6ba6a238157b
Remove unnecessary StartupTimeline.h inclusion r=chutten

Comment 3

3 months ago
bugherder
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.