Stop running framescript (listener.js) code when actors are enabled
Categories
(Remote Protocol :: Marionette, task, P1)
Tracking
(Fission Milestone:M7, firefox86 fixed)
Tracking | Status | |
---|---|---|
firefox86 | --- | fixed |
People
(Reporter: whimboo, Assigned: whimboo)
References
(Blocks 1 open bug)
Details
(Keywords: memory-footprint, Whiteboard: [marionette-fission-mvp])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Assignee | ||
Updated•4 years ago
|
Comment 1•4 years ago
|
||
Tracking marionette-fission-mvp bugs for Fission Beta milestone (M7)
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
This can wait until all jobs use actors by default, and we have the framescript fallback job. By doing that we have a complete coverage.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 3•4 years ago
|
||
All dependencies have been fixed. Now lets get the framescript not loaded when actors are enabled.
Assignee | ||
Comment 4•4 years ago
|
||
We will see a lot of memory usage improvements with that fix landed!
Assignee | ||
Comment 5•4 years ago
|
||
Assignee | ||
Comment 6•4 years ago
|
||
As discussed with Marco this bug should remain as MVP.
Comment 7•4 years ago
|
||
(In reply to Henrik Skupin (:whimboo) [⌚️UTC+2] (away 12/07 - 12/11) from comment #6)
As discussed with Marco this bug should remain as MVP.
If no resolution by end of day on Dec 20 then bug will be moved back into Reserve Backlog.
Comment 9•4 years ago
|
||
Backed out 1 changesets (bug 1669174) for WPT failures in uievents/click/auxclick_event.html. CLOSED TREE
Log:
https://treeherder.mozilla.org/logviewer?job_id=324591693&repo=autoland&lineNumber=14740
Push with failues:
https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&revision=d287bfb95caa1567d71868017661ddcb55ad0e3e
Backout:
https://hg.mozilla.org/integration/autoland/rev/914398229db8d83caf1e3501c3d760f800c8d1e9
Assignee | ||
Comment 10•4 years ago
|
||
This needs bug 1682757 to be fixed first.
Comment 11•4 years ago
|
||
Comment 12•4 years ago
|
||
Backed out for pointerevents failures - see Comment 8
Assignee | ||
Comment 13•4 years ago
|
||
The problem here is with the patch on bug 1682757.
Comment 14•4 years ago
|
||
Comment 15•4 years ago
|
||
bugherder |
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 16•4 years ago
|
||
The memory improvements here are actually:
Improvements:
Ratio Suite Test Platform Options Absolute values (old vs new)
6% Base Content JS macosx1014-64-shippable-qr 2,686,941.33 -> 2,518,160.00
6% Base Content JS linux1804-64-shippable 2,683,950.67 -> 2,515,744.00
6% Base Content JS windows10-64-shippable-qr 2,688,547.33 -> 2,519,948.00
6% Base Content JS windows10-64-shippable 2,688,513.33 -> 2,520,112.00
6% Base Content JS linux1804-64-shippable-qr 2,682,786.00 -> 2,515,292.00
2% Base Content Explicit windows10-64-shippable 8,546,901.33 -> 8,353,792.00
Assignee | ||
Updated•4 years ago
|
Updated•2 years ago
|
Description
•