Closed Bug 1270228 Opened 9 years ago Closed 5 years ago

about:debugging spawning child process in non-e10s mode

Categories

(DevTools :: about:debugging, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: bkelly, Unassigned)

References

(Blocks 1 open bug)

Details

STR: 1) Launch nightly in non-e10s mode. 2) Open about:memory and measure. Verify there is only a Main Process. 3) Open about:debugging in a tab. Switch to the worker panel. 4) Measure in about:memory again. Notice there is now a Web Content process with a single about:blank window. If you happen to have a service worker registered you may also see one of those works running in the child process in step 4. about:debugging should not force a child process in non-e10s mode.
Alex, it looks like about:debugging's Workers panel accidentally creates a Web Content process upon being loaded, but I'm not sure why. Is it possible that calling `client.mainRoot.listProcesses()` [0] or `client.getProcess()` creates a Web Content process if there are non? Or maybe somehow loading a process script like in [1] is to blame? [0] https://dxr.mozilla.org/mozilla-central/source/devtools/client/aboutdebugging/modules/worker.js#58 [1] https://dxr.mozilla.org/mozilla-central/source/devtools/server/actors/worker.js#332
Component: Developer Tools → Developer Tools: about:debugging
Flags: needinfo?(poirot.alex)
Priority: -- → P2
I don't think listProcesses nor getProcess create a process. But the loadProcessScript might be, did you tried commenting this line?
Flags: needinfo?(poirot.alex)
Flags: needinfo?(janx)
Flags: needinfo?(janx)
Priority: P2 → P3
Flags: needinfo?(janx)
Benoit, while you work on spawning or observing child processes in about:debugging, could you please keep an eye on this bug as well? (In non-e10s, we shouldn't spawn a child process, but apparently we do.)
Flags: needinfo?(janx) → needinfo?(bchabod)
(In reply to Jan Keromnes [:janx] from comment #3) > Benoit, while you work on spawning or observing child processes in > about:debugging, could you please keep an eye on this bug as well? (In > non-e10s, we shouldn't spawn a child process, but apparently we do.) Sure! I will try to reproduce, and see if I can isolate some code responsible for this bug.
Flags: needinfo?(bchabod)
Ben, sorry for the delay, I've been busy with other bugs. However, on latest Nightly, I can't manage to reproduce this bug. I tried different variations of the STR, with workers registered/running, without workers, with a lot of content pages opened, with about:debugging only, and in about:memory there is only the main process at each measure. Can you please confirm that you're able reproduce this bug, and maybe provide a more precise scenario if it's the case?
Flags: needinfo?(bkelly)
I see this: 1) Launch nightly 2) Disable e10s in options 3) Restart browser 4) Open example.com in a tab 5) Open about:memory in a tab 6) Minimize and measure memory 7) Observe a content process containing a single about:blank window. Eventually this process goes away. It seems unrelated to devtools in any case.
Flags: needinfo?(bkelly)
Product: Firefox → DevTools

Closing as non-e10s is gone

Status: NEW → RESOLVED
Closed: 5 years ago
tracking-e10s: - → ---
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.