Closed Bug 1645500 Opened 4 months ago Closed 4 months ago

Strip down process-content.js

Categories

(Toolkit :: General, task)

task

Tracking

()

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: kmag, Assigned: kmag)

References

(Blocks 1 open bug)

Details

(Keywords: perf-alert, Whiteboard: [overhead:10k])

Attachments

(2 files)

It's more complex than it needs to be, and stripping it down would save a couple of KB in each content process.

Ideally we would get rid of most of it and just use JSWindowActors instead of the current RemotePageManager model.

Attachment #9156390 - Attachment description: Bug 1645500: Strip down process-content.js to save memory. r=mccr8 → Bug 1645500: Part 1 - Strip down process-content.js to save memory. r=mccr8
Pushed by maglione.k@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5dd30b100700
Part 1 - Strip down process-content.js to save memory. r=mccr8
https://hg.mozilla.org/integration/autoland/rev/ca8139d58d58
Part 2 - Remove inner-window-destroyed observer from process-content.js. r=mccr8
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79

== Change summary for alert #26238 (as of Tue, 16 Jun 2020 15:41:26 GMT) ==

Improvements:

0.32% Base Content JS linux1804-64-shippable-qr opt 3,698,225.00 -> 3,686,448.33
0.31% Base Content JS linux1804-64-shippable opt 3,697,747.67 -> 3,686,335.67

For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=26238

Keywords: perf-alert
Whiteboard: [overhead:10k]
You need to log in before you can comment on or make changes to this bug.