1247258 - Use Array.prototype.includes in Telemetry code

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect, P1)

Description

[Georg Fritzsche [:gfritzsche]]

Use Array.prototype.includes in Telemetry code

Comment 1

[Georg Fritzsche [:gfritzsche]]

Attachment: 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.

Comment 2

[Georg Fritzsche [:gfritzsche]]

Attachment: Use Array.prototype.includes in Telemetry code

Comment 3

[Alessio Placitelli [:Dexter]]

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

Comment 4

[Georg Fritzsche [:gfritzsche]]

Attachment: Use Array.prototype.includes in Telemetry code

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/743097ee7470

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/743097ee7470