Open Bug 1811845 Opened 2 years ago Updated 2 years ago

Pass the browser toolbox scope to the server via TARGET_FILTERING_OPTIONS session data

Categories

(DevTools :: Framework, task)

task

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: ochameau, Assigned: ochameau, NeedInfo)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

For now, the server codebase doesn't really know if we are using a parent process only or a multiprocess browser toolbox/console.

Having an explicit field to distinguish the two would help implement a third mode where we focus only on the current tab's process (bug 1772310), but also enabling EFT on the browser toolbox (bug 1785266).

For the latter, this will actually help land the incremental refactoring in bug 1785106 by spawning only the parent process target actor from the server. And then bug 1785266 to tweak the browsing context filtering to only accept parent process browsing context.

This new data is passed down to all function filtering in/out the WindowGlobal's.
This will be used in a next patch to:

It would have been easier to re-use sessionContext and integrate toolbox scope in it...
But it would make sessionContext become mutable whereas it is static today.
It is built once on WatcherActor construction.
I think it is best to keep this data immutable. It can simplify the overall codebase
to have some environment data that can't change.
Otherwise, making sessionContext mutable would still require the same additional
JSWindowActor essage to update it accordingly in all processes and threads.

Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:ochameau, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Flags: needinfo?(poirot.alex)
Flags: needinfo?(jdescottes)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: