1815649 - image decoder threads are no longer named ImgDecoder so profiler doesn't pick them up

Status: NEW

(Core :: Gecko Profiler, defect, P3)

Description

[Timothy Nikkel (:tnikkel)]

When they got moved to taskcontroller they lost their name. The thread names now appear to be ex "TaskController #7". These threads aren't used exclusively for any one type of task anymore I take it? Should we just change the profiler ui to record TaskController threads when ImgDecoder threads are checked?

Comment 1

[Florian Quèze [:florian]]

TaskController is already listed, so I guess this means we can just remove the "ImgDecoder" checkbox. Bug 1746197 comment 3 had suggestions for new thread names that might be worth listing. I would personally vote for "IPC I/O" as any delay on these threads can quickly cause significant issues elsewhere.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1672597

:bas.schouten, since you are the author of the regressor, bug 1672597, could you take a look? Also, could you set the severity field?

For more information, please visit auto_nag documentation.

Comment 3

[Timothy Nikkel (:tnikkel)]

Probably don't need to needinfo Bas for this.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1672597