Closed Bug 1732120 Opened 3 years ago Closed 3 years ago

Rename BrowsingContextTargetActor WindowGlobalTargetActor

Categories

(DevTools :: General, task)

task

Tracking

(firefox94 fixed)

RESOLVED FIXED
94 Branch
Tracking Status
firefox94 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Since we enable server side targets, all BrowsingContextTargetActor are actually rather WindowGlobalTargetActor as they all follow the lifecycle of the WindowGlobal.

So that it would be cleared if we rename this class as well as anything refering to this particular target (front, typeName, spec,...)

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

This intermediate actor is no longer useful for a while.
(Since we dropped the LocalTabProxy. I think since we introduced the TabDescriptor?)

I updated actor-hierarchy.md significantly as it was quite outdated!
We would probably benefit having a section dedicated to the WatcherActor.

Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/23fa337d8f5a
[devtools] Rename BrowsingContextTargetActor to WindowGlobalTargetActor. r=nchevobbe,devtools-backward-compat-reviewers
https://hg.mozilla.org/integration/autoland/rev/1f92bcc65877
[devtools] Remove FrameTargetActor in favor of WindowGlobalTargetActor. r=nchevobbe,devtools-backward-compat-reviewers
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 94 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: