Closed Bug 1520774 Opened 10 months ago Closed 10 months ago

Remove TargetFactory.forWorker/forRemoteTab

Categories

(DevTools :: Framework, enhancement, P2)

enhancement

Tracking

(firefox67 fixed)

RESOLVED FIXED
Firefox 67
Tracking Status
firefox67 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

(Whiteboard: dt-fission)

Attachments

(2 files, 2 obsolete files)

Once Target fronts and Target class are merged, TargetFactory.{forWorker,forRemoteTab} are no longer necessary as the target instances are returned by the Front methods exposing target fronts like RootFront.getTab.

TargtetFactory.forTab is still relevant as it help creating the server and client and directly accept a xul:tab as input, which is handy for tests and production code.
We might want to unify all codes starting a server/client. That may be target factory or some other module.

Depends on D18537

Depends on D18539

Attachment #9041134 - Attachment is obsolete: true
Attachment #9041135 - Attachment is obsolete: true
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/023ecfc5cf46
Remove TargetFactory.forRemoteTab. r=yulia
https://hg.mozilla.org/integration/autoland/rev/a42ea7bbd583
Remove TargetFactory.forWorker. r=yulia
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
You need to log in before you can comment on or make changes to this bug.