Closed Bug 1574343 Opened 5 years ago Closed 5 years ago

Improve process labels and icons

Categories

(DevTools :: Debugger, enhancement, P1)

enhancement

Tracking

(firefox70 fixed)

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: jlast, Assigned: jlast)

References

(Blocks 1 open bug)

Details

(Whiteboard: dt-fission-m1)

Attachments

(3 files)

We currently show Processes as if they are workers in the sources tree and as if they are threads in the Threads pane. Neither case is it ideal :).

  • We should show a better label, perhaps based off of a title or url.
  • We should use a different icon, perhaps based off a favicon or other metadata we have.

We could also just do something generic like "Content Process 2" with a gear or something if we do not want to invest too much into heuristics.

Here's what we currently have in the browser toolbox if you enable the browsertoolbox fission pref.

I'm curious what others think would some good heuristics for a process label and icon in the sources tree?

Flags: needinfo?(ystartsev)
Flags: needinfo?(spenades)
Flags: needinfo?(poirot.alex)
Flags: needinfo?(odvarko)
Flags: needinfo?(hkirschner)
Priority: -- → P2
Blocks: dbg-fission
No longer blocks: dbg-watchpoints

Whatever decisions we make for the process label and icon should also apply for the threads pane

Summary: Improve the Source Tree's Process label and icon → Improve process labels and icons
Whiteboard: dt-fission

You could potentially use the content process id -- we don't have any other identifying information for those processes... So for example "Content Process <id>". Regarding the icons -- I think talking to Victoria would be the best idea

Flags: needinfo?(ystartsev)
Assignee: nobody → jlaster
Status: NEW → ASSIGNED
Priority: P2 → P1
Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/002962ee07d0
Improve process labels and icons. r=davidwalsh,yulia
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
Flags: needinfo?(spenades)
Flags: needinfo?(poirot.alex)
Flags: needinfo?(odvarko)

clearing ni

Flags: needinfo?(hkirschner)
Whiteboard: dt-fission → dt-fission dt-fission-m1
Whiteboard: dt-fission dt-fission-m1 → dt-fission-m1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: