Closed Bug 1472163 Opened 3 years ago Closed 3 years ago
Remove sync/async blocklist load probe
59 bytes, text/x-review-board-request
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 email@example.com: https://hg.mozilla.org/integration/autoland/rev/78472262694a Removed histogram from Histograms.json r=Gijs
You need to log in before you can comment on or make changes to this bug.