1444490 - Mark netwerk classes final

Status: RESOLVED FIXED

(Core :: Networking, defect, P5)

Description

[Tom Ritter [:tjr]]

Bug 1443252 showed a dramatic improvement in build times by marking just two classes final. While that was definitely the biggest low-hanging fruit, there are other fruit to be plucked, including quite a bit in netwerk/

Every class marked final here will de-virtualize at least 10 method calls. And the single member (SetUsername) will devirtualize 26 calling instances.

The class changes in particular should show some build time improvement, and in general we gain a tiny amount of performance from this.

Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=011081864cb5ec5cc9cabd9aa51ca12b49ac19d7&selectedJob=167024590

I did not queue up a ton of Talos runs for an in-depth comparison, but it seems like there are definite build time improvements, and small perf improvements with no expected regressions.

Comment 1

[Tom Ritter [:tjr]]

Attachment: Bug 1444490 Declare classses (and one method) in network/ final to reduce virtual function calls

Review commit: https://reviewboard.mozilla.org/r/226532/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/226532/

Comment 2

[u408661]

Comment on attachment 8957640 [details]
Bug 1444490 Declare classses (and one method) in network/ final to reduce virtual function calls

https://reviewboard.mozilla.org/r/226532/#review233276

LGTM. Sorry for the lag.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ab89c04fa91c
Declare classses (and one method) in network/ final to reduce virtual function calls r=nwgh

Comment 4

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/ab89c04fa91c