Closed Bug 1247258 Opened 8 years ago Closed 8 years ago

Use Array.prototype.includes in Telemetry code

Categories

(Toolkit :: Telemetry, defect, P1)

defect
Points:
1

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: gfritzsche, Assigned: gfritzsche)

Details

(Whiteboard: [measurement:client])

Attachments

(1 file, 2 obsolete files)

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.
Attachment #8717883 - Flags: review?(alessio.placitelli)
Points: --- → 1
Priority: -- → P1
Whiteboard: [measurement:client]
Attachment #8717884 - Flags: review?(alessio.placitelli)
Attachment #8717883 - Attachment is obsolete: true
Attachment #8717883 - Flags: review?(alessio.placitelli)
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
Attachment #8717891 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/743097ee7470
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: