1388325 - Stop getting telemetry for deprecated feature in addons (JS_DEPRECATED_LANGUAGE_EXTENSIONS_IN_ADDONS)

Status: RESOLVED FIXED

(Core :: JavaScript Engine, enhancement, P3)

Description

[Tooru Fujisawa [:arai]]

given that so many legacy APIs are getting removed in bug 1347507,
it might be nice to just stop getting telemetry for deprecated feature in addons and just focus on web content usage.

currently we're tracking the usage of ForEach, LegacyGenerator, ExpressionClosure, and BlockScopeFunRedecl.

for-each is disabled in bug 1083470.

legacy generator is available only in versioned JS, and non-legacy addons shouldn't use it (am I correct?)

the count for BlockScopeFunRedecl is almost 0.

there might be some chance that expression closure is used in non-legacy addons tho...

Comment 1

[Masatoshi Kimura [:emk]]

(In reply to Tooru Fujisawa [:arai] from comment #0)
> legacy generator is available only in versioned JS, and non-legacy addons
> shouldn't use it (am I correct?)

Correct. we are actively preventing WebExtensions from using versioned JS (bug 1213632).

Comment 2

[Tooru Fujisawa [:arai]]

Attachment: Stop getting telemetry for deprecated feature in addons (JS_DEPRECATED_LANGUAGE_EXTENSIONS_IN_ADDONS)

almost a backout of bug 1211164.

Comment 3

[Jan de Mooij [:jandem]]

Comment on attachment 8930081 [details] [diff] [review]
Stop getting telemetry for deprecated feature in addons (JS_DEPRECATED_LANGUAGE_EXTENSIONS_IN_ADDONS)

Review of attachment 8930081 [details] [diff] [review]:
-----------------------------------------------------------------

Do you think this could be useful in the future for things like toSource and Array/String generics in web extensions?

r=me either way though.

Comment 4

[Tooru Fujisawa [:arai]]

Thank you for reviewing :)

(In reply to Jan de Mooij [:jandem] from comment #3)
> Do you think this could be useful in the future for things like toSource and
> Array/String generics in web extensions?

we're already run out of buckets in that probe (bug 1331909, bug 1339777)
so the probe itself is not reusable.

the the change in JSCompartment methods can be reused once we decide to collect telemetry for webextensions.

Comment 5

[Jan de Mooij [:jandem]]

Ah, makes sense!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/966a8cda3001
Stop getting telemetry for deprecated feature in addons (JS_DEPRECATED_LANGUAGE_EXTENSIONS_IN_ADDONS) r=jandem

Comment 7

[Tooru Fujisawa [:arai]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/966a8cda3001a83edbc796accabd79094f155f61
Bug 1388325 - Stop getting telemetry for deprecated feature in addons (JS_DEPRECATED_LANGUAGE_EXTENSIONS_IN_ADDONS) r=jandem

Comment 8

[Tiberius Oros[:tiberius_oros]]

https://hg.mozilla.org/mozilla-central/rev/966a8cda3001