1814210 - Convert New Tab Page to use actor based RPM

Status: RESOLVED FIXED

(Firefox :: New Tab Page, task)

Description

[Neil Deakin]

The new tab/activity stream page is the only remaining consumer of the old message port based RemotePageManager (in toolkit/components/remotepagemanager).

This bug is about converting it to the jsactor-based implementation (toolkit/actors/RemotePageChild.jsm)

Comment 1

[Neil Deakin]

Attachment: Bug 1814210, switch ActivityStreamMessageChannel to use actor-based Remote Page Manager, r?mardak

Comment 2

[Neil Deakin]

Attachment: Bug 1814210, remove remote page manager now that it is no longer used, r=mossop

Depends on D171604

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3ae01032459a
switch ActivityStreamMessageChannel to use actor-based Remote Page Manager, r=Mardak

Comment 5

[Sandor Molnar[:smolnar]]

Backed out for causing bc failures in browser/base/content/test/static/browser_all_files_referenced.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/ea26e119f36505581f5d2da96d74fbca00286305

Push with failures

Failure log -> TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | unreferenced file: resource://gre/modules/remotepagemanager/RemotePageManagerParent.sys.mjs

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/980bbb2d3e06
remove remote page manager now that it is no longer used, r=mossop

Comment 7

[Sandor Molnar[:smolnar]]

Backed out for causing multiple dt/bc failures in modules/remotepagemanager/RemotePageManagerParent.sys.mjs

Backout link: https://hg.mozilla.org/integration/autoland/rev/52acca838d2b20f16a45ea4c45d9fd4ff9a6c3f9

Push with failures

Failure log dt -> TEST-UNEXPECTED-FAIL | devtools/client/responsive/test/browser/browser_exit_button.js | A promise chain failed to handle a rejection: Failed to load resource://gre/modules/remotepagemanager/RemotePageManagerParent.sys.mj

Failure log bc ->browser/base/content/test/about/browser_aboutHome_search_composing.js | Uncaught exception in test bound - undefined - timed out after 50 tries

Comment 8

[Neil Deakin]

Going to check both patches in at once this time.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/36041facd750
switch ActivityStreamMessageChannel to use actor-based Remote Page Manager, r=Mardak
https://hg.mozilla.org/integration/autoland/rev/7dbe5d77658b
remove remote page manager now that it is no longer used, r=mossop

Comment 10

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/36041facd750
https://hg.mozilla.org/mozilla-central/rev/7dbe5d77658b