Closed Bug 1510806 Opened 6 years ago Closed 5 years ago

[meta] Make RDM Fission-ready

Categories

(DevTools :: Responsive Design Mode, enhancement, P3)

enhancement

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1574885

People

(Reporter: mconley, Unassigned)

References

(Blocks 2 open bugs)

Details

(Keywords: meta)

It looks like RDM currently closes the UI when the process flips for a tab. With Fission, we should expect many process flips to occur. We should probably make RDM resilient to that.
Agreed we need to fix this for Fission. Is there a convenient way to force a process flip, for purposes of testing?
Flags: needinfo?(mconley)
Priority: -- → P3
(In reply to Brad Werth [:bradwerth] from comment #1) > Agreed we need to fix this for Fission. Is there a convenient way to force a > process flip, for purposes of testing? At this stage, yes you can[1], but you might want to connect with Nika from DOM to see how they're planning on re-architecting our process-flipping stuff for Fission. I think she's planning on putting most of that logic somewhere in the networking layer in the parent process. Nika, do you have any published plans anywhere on how process flipping is going to work in the future that bradwerth can look at? [1]: Here's one way of doing it with the current infrastructure on Nightly: 1. Visit https://www.mozilla.org 2. Enter RDM 3. Browse to about:newtab ER: RDM should stay open AR: RDM closes and about:newtab is loaded in the full page.
Flags: needinfo?(mconley)
ni? to nika for comment 2.
Flags: needinfo?(nika)
Other pages which you can load to trigger a swap include: 1. Loading file:// URIs 2. Loading moz-extension:// URIs 3. Loading about:config or other chrome-only pages 4. Loading about:newtab (I think nightly only) 5. Switching from any of these URIs to any other of these URIs. IIRC the way that RDM works is by placing a chrome document inside the xul:browser, and forwarding messages from the outer xul:browser into an <iframe mozbrowser> internally. When we do a process switch we currently trigger that switch by replacing the xul:browser element in the DOM. As we move to a new system for doing process swaps without replacing the browser node for fission, this may be changed. It may not be worthwhile to fix the issue for the current system of process switching given the changes coming down the pipe.
Flags: needinfo?(nika)
Alias: dt-fission-rdm
Summary: Make Responsive Design Mode survive process flips → [meta] Make RDM Fission-ready
Keywords: meta
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
Alias: dt-fission-rdm
You need to log in before you can comment on or make changes to this bug.