Closed
Bug 1520774
Opened 5 years ago
Closed 5 years ago
Remove TargetFactory.forWorker/forRemoteTab
Categories
(DevTools :: Framework, enhancement, P2)
DevTools
Framework
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.
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
Depends on D18537
Assignee | ||
Comment 3•5 years ago
|
||
Assignee | ||
Comment 4•5 years ago
|
||
Depends on D18539
Updated•5 years ago
|
Attachment #9041134 -
Attachment is obsolete: true
Updated•5 years ago
|
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
Comment 6•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/023ecfc5cf46
https://hg.mozilla.org/mozilla-central/rev/a42ea7bbd583
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
You need to log in
before you can comment on or make changes to this bug.
Description
•