Closed Bug 1820243 Opened 2 years ago Closed 2 years ago

Transition about:newtab/about:home/about:welcome to remote page actor instead of legacy RemotePageManager

Categories

(Firefox :: New Tab Page, defect)

Desktop
All
defect

Tracking

()

RESOLVED DUPLICATE of bug 1814210
Tracking Status
firefox112 --- affected

People

(Reporter: Gijs, Unassigned)

Details

bug 1614713 moved all consumers of RemotePageManager off of RemotePageManager, and into actor classes that extend RemotePageChild or RemotePageParent.

... all except one! about:newtab/about:home/about:welcome is still holding out on the old infrastructure.

At the moment, this is part of what's requiring us to instantiate a BrowserTab actor for each and every page load. I'm addressing that part in bug 1718082, but I'm using a stopgap solution by moving the forced initialization of RPM into the privileged about content process and into the existing ASRouter actor. I haven't tried to touch all the messaging infrastructure itself.

I think it should be reasonably straightforward to:

  1. make the ASRouter actor pair inherit from RemotePageChild/Parent
  2. switch the 3 or so messages that it sends over the message port to use the "normal" RPM infrastructure for this instead
  3. then remove the legacy RemotePageManager code entirely.

This bug is here to do that work. :-)

I already did all this for bug 1814210, but never had time to polish the patch. I'll post it in that bug.

(In reply to Neil Deakin from comment #1)

I already did all this for bug 1814210, but never had time to polish the patch. I'll post it in that bug.

Ah, drat, I didn't spot that bug when looking for it. Thank you! I'll dupe this over.

Status: NEW → RESOLVED
Closed: 2 years ago
Duplicate of bug: 1814210
Resolution: --- → DUPLICATE
No longer blocks: 1614713
No longer depends on: 1718082
You need to log in before you can comment on or make changes to this bug.