After we stop sending `childPayloads` with bug 1443599, we should remove the childPayload recording code, and any code that would display it in about:telemetry.

Comment on attachment 8964949 [details] [diff] [review] Remove legacy Telemetry component `childPayloads` Review of attachment 8964949 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/components/telemetry/Histograms.json @@ -7204,5 @@ > "high": 30, > "n_buckets": 29, > "description": "The size (MB) of the ping data submitted to Telemetry exceeding the maximum size" > }, > - "TELEMETRY_DISCARDED_CONTENT_PINGS_COUNT": { Good catch. ::: toolkit/components/telemetry/TelemetrySession.jsm @@ -1670,5 @@ > }, > > - sendContentProcessPing: function sendContentProcessPing(reason) { > - this._log.trace("sendContentProcessPing - Reason " + reason); > - const isSubsession = !this._isClassicReason(reason); With the removal of saved-session and this, we're -this- close to removing _isClassicReason. Pretty sure we "just" need to rework how we want to deal with that one test and how we gather the current information to show in about:telemetry. ::: toolkit/content/aboutTelemetry.xhtml @@ -125,5 @@ > </div> > </div> > - <div> > - <h4 class="title">&aboutTelemetry.payloadChoiceHeader;</h4> > - <select id="choose-payload"></select> So happy to see this go.

Pushed by nerli@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/4dfc990e1b9c Remove legacy Telemetry component `childPayloads` r=chutten