Open Bug 1266561 Opened 8 years ago Updated 2 years ago

isRootActor should be split up into meaningful settings

Categories

(DevTools :: General, enhancement, P3)

Product:
DevTools
Component:
General
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: jryans, Unassigned)

References

(Blocks 1 open bug)

Details

ChromeActor (and soon WindowActor being added for positron) set "isRootActor"[1] to triggers different paths in certain actors for the type of windows they hold.

However, the name "isRootActor" does not really convey any information, and it appears to be mean different things for the actors that test it.  We should change this to one or more clearly named flags.

[1]: https://dxr.mozilla.org/mozilla-central/search?q=isRootActor&case=true
Product: Firefox → DevTools

Julian, is this still relevant?

Honza

Flags: needinfo?(jdescottes)

I think so, we still define isRootActor on a few target actors (parent process, content process, webextension), and use it for unclear reasons.
A cleanup would be nice.

Flags: needinfo?(jdescottes)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.