1569238 - Debugger should show sources for ContentProcessTargets

Status: RESOLVED DUPLICATE of bug 1573315

(DevTools :: Debugger, enhancement, P1)

Description

[Yulia Startsev [:yulia]]

Attachment: Screenshot 2019-07-26 at 18.31.48.png

For the debugger, we need to be able to show other processes than the main process in the UI. We currently have a ui for workers but not the processes.

Comment 1

[:Harald Kirschner :digitarald]

Clarifying the title, as the Sources panel has UI already to show worker sources, which can be tested with http://dbg-workers.glitch.me/

Comment 2

[Brian Hackett [Laid off!]]

FWIW the intent with the windowless workers UI was that it should be forward-compatible with fission threads. Main threads for other content processes would show up in the UI as if they were worker threads, they just need new names and the plumbing required to supply their sources, pause state, etc. to the debugger.

Comment 3

[Jason Laster [:jlast]]

Yulia, do you have a test case in mind?

Also, what do you think would need to be done in terms of listing to the other processes?

Comment 5

[Yulia Startsev [:yulia]]

Attachment: content processes are listed

Comment 6

[Yulia Startsev [:yulia]]

Attachment: Sources are found

Comment 7

[Jason Laster [:jlast]]

Closing as the debugger now shows content processes and their sources in the sources tree. We will need follow up bugs to improve the UI, but they now behave similar to workers.