Remove the reporting of Telemetry Experiments (ie, the ones overseen by the now-unused browser/experiments/Experiments.jsm module) from the Telemetry Environment, but leave normandy experiments (overseen by PreferenceExperiments.jsm) alone. Then disable any tests and update any documentation. Reporting seems to just be in _getActiveExperiment in TelemetryEnvironment, so the simplest path may be to have that always return {}. There are quite a few test files that use Experiments and will need changing. We have a decent amount of documentation on Experiments that will need to be changed.

I disabled `_getActiveExperiment` as mentioned. The documentation on that part was limited. There's more experiments documentation, e.g. [1] and [2] [1]: https://firefox-source-docs.mozilla.org/browser/experiments/experiments/index.html [2]: https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/collection/experiments.html Am I correct that these belong to the normandy experiments? It's not clear from the name as "normandy" is never mentioned.

Comment on attachment 8961445 [details] [diff] [review] Stop reporting legacy Telemetry component `Telemetry Experiments` Review of attachment 8961445 [details] [diff] [review]: ----------------------------------------------------------------- Looks good for stopping the reporting. This one I predict will be incredibly satisfying to rip out completely.

(In reply to Jan-Erik Rediger [:janerik] from comment #1) > Created attachment 8961445 [details] [diff] [review] > Stop reporting legacy Telemetry component `Telemetry Experiments` > > I disabled `_getActiveExperiment` as mentioned. > The documentation on that part was limited. > > There's more experiments documentation, e.g. [1] and [2] > > [1]: > https://firefox-source-docs.mozilla.org/browser/experiments/experiments/ > index.html > [2]: > https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/ > telemetry/collection/experiments.html > > Am I correct that these belong to the normandy experiments? > It's not clear from the name as "normandy" is never mentioned. [1] Can definitely be removed. And its child page. [2] Must be maintained, as it's the way that others (like normandy/shield/etc) annotate the environment with a list of active experiments

Will remove the documentation in the follow-up patch then together with the code.

Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d8480fd61e80 Stop reporting legacy Telemetry component `Telemetry Experiments`. r=chutten