59 bytes, text/x-review-board-request
If a ping has environment data, EnvironmentData.render(ping) will render it. It will then use special rendering to render the Addon data. If the Environment block doesn't have an Addon section (due to, among other things, delayed addon init) then it defaults to showing the Raw JSON of the ping. With the new about:telemetry redesign, showing the Raw JSON involves opening a new tab to show it using Firefox's spiffy JSON viewer. This is jarring and avoidable. So, instead, _check_ that an addons subsection exists before rendering one.
Attachment #8904675 - Flags: review?(gfritzsche) → review?(alessio.placitelli)
Comment on attachment 8904675 [details] bug 1396894 - Don't error out when rendering normally-structured pings https://reviewboard.mozilla.org/r/176478/#review181840 Took me a bit to figure the "key" change :S Looks good!
Attachment #8904675 - Flags: review?(alessio.placitelli) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/378fb9246039 Don't error out when rendering normally-structured pings r=Dexter
You need to log in before you can comment on or make changes to this bug.