Remove sync/async blocklist load probe

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P3
normal
RESOLVED FIXED
Last year
Last year

People

(Reporter: Gijs, Assigned: lpoflynn, Mentored)

Tracking

({good-first-bug})

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

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

Attachments

(1 attachment)

Priority: -- → P3
Whiteboard: [good first bug][lang=json]
(In reply to :Gijs (he/him) from comment #0)
> Bug 1454378 removed all the code that recorded this telemetry, because we no
> longer ever load the list synchronously. We should remove the histogram from
> Histograms.json . This involves removing this block of JSON:
> 
> https://searchfox.org/mozilla-central/rev/
> d2966246905102b36ef5221b0e3cbccf7ea15a86/toolkit/components/telemetry/
> Histograms.json#208-215

Oops, fixed link:
https://searchfox.org/mozilla-central/rev/d2966246905102b36ef5221b0e3cbccf7ea15a86/toolkit/components/telemetry/Histograms.json#208-215
Hi! I would like to acquire some experience contributing to the Mozilla codebase by taking this issue. Could I be assigned?
(In reply to Liam from comment #2)
> Hi! I would like to acquire some experience contributing to the Mozilla
> codebase by taking this issue. Could I be assigned?

Sure! We generally assign first bugs after you put up your first patch. In case you need help getting started:

https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Introduction
https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Simple_Firefox_build

tells you how to get the source and build it. Feel free to ask questions on here if you get stuck. I'm traveling this weekend so you may not get a response from me immediately - apologies in advance - but you could also try asking on IRC in #introduction (there's a link in that first link) where other people may be around who are able to help. :-)
Comment on attachment 8988955 [details]
Removed histogram from Histograms.json (Bug 1472163)

https://reviewboard.mozilla.org/r/254064/#review260828

Looks great, thank you!
Attachment #8988955 - Flags: review?(gijskruitbosch+bugs) → review+
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/78472262694a
Removed histogram from Histograms.json r=Gijs
Assignee: nobody → lpoflynn
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/78472262694a
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.