Doing some measurements, it looks like there are some easy small wins with MessageIndent, MessageRepeat, and MessageIcon

Comment on attachment 8866629 [details] Bug 1363768 - Prevent creating constant elements during render; https://reviewboard.mozilla.org/r/138234/#review141464 Just a small thing, but R+ if TRY and mocha are green ::: devtools/client/webconsole/new-console-output/components/message.js:194 (Diff revision 1) > ))); > } else { > notesNodes = []; > } > > - const repeat = MessageRepeat({repeat: this.props.repeat}); > + const repeat = this.props.repeat ? MessageRepeat({repeat: this.props.repeat}) : null; IF we're doing this here, we can go a little further and do not print anything if repeat is < 2 ``` const repeat = this.props.repeat && this.props.repeat > 1 ? MessageRepeat({repeat: this.props.repeat}) : null; ``` It also means that we can remove tis logic from the MessageRepeat function

Thanks, good points. Updated, and pushed to try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4e3fe814e311328fe69e35702eaad0f0d6f3ef04

Pushed by bgrinstead@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bea7660031a0 Prevent creating constant elements during render;r=nchevobbe