Remove unused probe NEWTAB_PAGE_LIFE_SPAN_SUGGESTED

RESOLVED FIXED in Firefox 58

Status

()

P3
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: chutten, Assigned: amorris2003, Mentored)

Tracking

Trunk
mozilla58
Points:
---

Firefox Tracking Flags

(firefox58 fixed)

Details

(Whiteboard: [good first bug][lang=js])

Attachments

(1 attachment)

(Reporter)

Description

a year ago
It should've been removed as a part of bug 1241390 which removed the suggested tiles feature.

Remove from the following locations:
toolkit/components/telemetry/Histograms.json
toolkit/components/telemetry/histogram-whitelists.json

Then ensure that ./mach eslint -wo succeeds and you can still build Firefox.
(Assignee)

Comment 1

a year ago
Created attachment 8918531 [details] [diff] [review]
1408134.patch
Attachment #8918531 - Flags: review?(chutten)
(Reporter)

Updated

a year ago
Attachment #8918531 - Flags: review?(chutten) → review+
(Reporter)

Comment 2

a year ago
Thanks, this looks great! I've marked it for push into the tree. With luck your contribution will be in the next build of Firefox Nightly.

Would you like any help finding more bugs to fix?
Assignee: nobody → amorris2003
Status: NEW → ASSIGNED
Keywords: checkin-needed

Comment 3

a year ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/05cd36743f31
Remove unused probe NEWTAB_PAGE_LIFE_SPAN_SUGGESTED. r=chutten
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/05cd36743f31
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox58: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
No longer depends on: 1409411
You need to log in before you can comment on or make changes to this bug.