1710478 - aboutProcesses.js' displayDOMWindowRow function provides too many fluent arguments

Status: RESOLVED FIXED

(Toolkit :: Performance Monitoring, defect)

Description

[Florian Quèze [:florian]]

When profiling about:processes after bug 1705827 landed, I see that JSON.stringify-ing and parsing of fluent arguments takes a significant part of the remaining time.

Before digging further into this, I think it makes sense to reduce the size of the data being serialized and parsed, by removing the fluent arguments that are unused.

My patch in bug 1710441 does that for process rows. It makes sense to also do it for rows were tabs are listed (the url, which is potentially long, is currently included unconditionnaly, but is only used in the subframes case).

Comment 1

[Florian Quèze [:florian]]

Attachment: Bug 1710478 - only provide the fluent arguments that will be used when displaying tab rows in about:processes, r=dthayer.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/71c52ecc7fa7
only provide the fluent arguments that will be used when displaying tab rows in about:processes, r=dthayer.

Comment 3

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/71c52ecc7fa7