PreviousExperimentAddon.blocklistState will always return undefined

RESOLVED FIXED in Firefox 59

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: standard8, Assigned: leakey94, Mentored)

Tracking

(Blocks 1 bug)

Trunk
mozilla59
Points:
---

Firefox Tracking Flags

(firefox58 wontfix, firefox59 fixed)

Details

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

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
In starting to look at turning on no-unused-expressions, I noticed this in Experiments.jsm:

http://searchfox.org/mozilla-central/rev/dca019c94bf3a840ed7ff50261483410cfece24f/browser/experiments/Experiments.jsm#2264-2266

  get blocklistState() {
    Ci.nsIBlocklistService.STATE_NOT_BLOCKED;
  },

That function will always return undefined. I can't see an obvious case of this breaking something at the moment, but it could do.
All for want of the word "return"

There are currently no Telemetry Experiments being run, so this can't cause problems too easily. But it's an easy fix and a good first bug.
Mentor: chutten
Priority: -- → P3
Whiteboard: [good first bug][lang=js]

Comment 2

2 years ago
Can I take this one? I just started looking for bugs and this seems as good as any to make my first contribution.
Please do! I have assigned the bug to you. Do let me know if you have any questions.
Assignee: nobody → vincentdhilton
Status: NEW → ASSIGNED
Hello Vincent, are you still looking at this bug? Do you need any help?
Flags: needinfo?(vincentdhilton)
(Assignee)

Comment 5

a year ago
If Vincent is no longer looking at this bug, I'm happy to fix it.
Flags: needinfo?(chutten)
Please, by all means do.

Vincent, if you'd like assistance choosing a different first bug, let me know here or on #telemetry on irc.mozilla.org and I can help you out (https://wiki.mozilla.org/Irc)
Assignee: vincentdhilton → leakey94
Flags: needinfo?(vincentdhilton)
Flags: needinfo?(chutten)
Comment hidden (mozreview-request)

Comment 8

a year ago
mozreview-review
Comment on attachment 8930471 [details]
Bug 1409694 - Add return statement for PreviousExperimentAddon.blocklistState.

https://reviewboard.mozilla.org/r/201622/#review206986

Looks good!
Attachment #8930471 - Flags: review?(chutten) → review+
Thank you for your contribution, Ryan! I've marked it for checkin so it should be in Nightly in the next couple of days.

Would you like help finding the next bug to tackle (maybe one of these? https://www.joshmatthews.net/bugsahoy/?internals=1) or would you like to search around yourself?
Keywords: checkin-needed
(Assignee)

Comment 10

a year ago
(In reply to Chris H-C :chutten from comment #9)
> Thank you for your contribution, Ryan! I've marked it for checkin so it
> should be in Nightly in the next couple of days.
> 
> Would you like help finding the next bug to tackle (maybe one of these?
> https://www.joshmatthews.net/bugsahoy/?internals=1) or would you like to
> search around yourself?

Perfect, thanks Chris! I'd love some help finding another bug to tackle, I'm wanting to move on from "good first bugs" but am finding that there's quite a leap in difficulty as a beginner.
Flags: needinfo?(chutten)
It's true, we do make a special effort to make the good first bug as easy as possible (because learning the tools and process are hard enough without a difficult bug making it impossible :)

good next bugs/good second bugs are harder to come by. It's hard to strike that balance. Let's see what I can find you.

bug 1278247 is about renaming files. Not too complex, but moreso than adding "return" to one line.
bug 1289094 is really chewy. It involves messing with some JS <-> C++ APIs 
bug 1290256 involves a little bit about networking.
bug 1301365 is also on the chewy side. This will require reading some Python, and adding capabilities in C++

I'm not sure what your language strengths are, so I went for a mix. Hopefully there's something in here that catches your fancy.
Flags: needinfo?(chutten)
(Assignee)

Comment 12

a year ago
(In reply to Chris H-C :chutten from comment #11)
> It's true, we do make a special effort to make the good first bug as easy as
> possible (because learning the tools and process are hard enough without a
> difficult bug making it impossible :)
> 
> good next bugs/good second bugs are harder to come by. It's hard to strike
> that balance. Let's see what I can find you.
> 
> bug 1278247 is about renaming files. Not too complex, but moreso than adding
> "return" to one line.
> bug 1289094 is really chewy. It involves messing with some JS <-> C++ APIs 
> bug 1290256 involves a little bit about networking.
> bug 1301365 is also on the chewy side. This will require reading some
> Python, and adding capabilities in C++
> 
> I'm not sure what your language strengths are, so I went for a mix.
> Hopefully there's something in here that catches your fancy.

Much appreciated, I'm sure I can make some contributions in there.

Comment 13

a year ago
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a75234ffae56
Add return statement for PreviousExperimentAddon.blocklistState. r=chutten
Keywords: checkin-needed

Comment 14

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a75234ffae56
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Do we want this uplifted to 58?
Flags: needinfo?(chutten)
It's not harming anything (Telemetry Experiments are presently unused and are being removed (bug 1415284)), so I don't think there's any need.
Flags: needinfo?(chutten)
You need to log in before you can comment on or make changes to this bug.