Use Array.prototype.includes in Telemetry code
We have Array.prototype.includes since Fx 43, which is much more readable than the previous a.indexOf(b) based idiom.
Points: --- → 1
Priority: -- → P1
Attachment #8717884 - Flags: review?(alessio.placitelli) → review+
There's only one occurrence in the tests, in case you want to take care of that one as well: https://dxr.mozilla.org/mozilla-central/source/toolkit/components/telemetry/tests/unit/test_TelemetryEnvironment.js#409
Attachment #8717884 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.