Closed
Bug 1522571
Opened 6 years ago
Closed 5 years ago
Move AddonTargetFront outside of devtools/shared/front/targets
Categories
(DevTools :: Framework, enhancement, P1)
DevTools
Framework
Tracking
(firefox70 fixed)
RESOLVED
FIXED
Firefox 70
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: ochameau, Assigned: yulia)
References
(Blocks 1 open bug)
Details
(Whiteboard: dt-fission-m1)
Attachments
(1 file)
This class is no longer a Target class. Instead, its connect() method allows returning one.
So that this class no longer belongs to the targets directory.
Assignee | ||
Comment 1•5 years ago
|
||
This might be described as a descriptor?
Assignee | ||
Updated•5 years ago
|
Blocks: dt-fission-framework
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → ystartsev
Priority: P2 → P1
Updated•5 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•5 years ago
|
||
It turns out that the "addonTargetFront" was being used by the "WebExtensionActor". I aligned the naming and changed the spec to reflect the relationship between these two -- which has no attach, detach, reconfigure methods as mentioned by alex. Ultimately, this should probably be turned into a descriptor class.
Assignee | ||
Comment 3•5 years ago
|
||
Pushed by ystartsev@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/586631218ed0
Unify addonTargetFront with webExtensionSpec; r=ochameau
Comment 5•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
Updated•5 years ago
|
Whiteboard: dt-fission → dt-fission dt-fission-m1
Updated•3 years ago
|
Whiteboard: dt-fission dt-fission-m1 → dt-fission-m1
You need to log in
before you can comment on or make changes to this bug.
Description
•