Closed
Bug 1590401
Opened 5 years ago
Closed 5 years ago
Refactor Front.onFront/offFront/onFrontDestroyed/offFrontDestroyed into Front.watchFronts/unwatchFronts
Categories
(DevTools :: Framework, enhancement, P3)
DevTools
Framework
Tracking
(firefox72 fixed)
RESOLVED
FIXED
Firefox 72
| Tracking | Status | |
|---|---|---|
| firefox72 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
Details
Attachments
(1 file)
Bug 1471754 is introducing a new API: TargetList.watchTargets/unwatchTargets.
This API is actually very similar to Front.onFront/offFront/onFrontDestroyed/offFrontDestroyed and there is no good reason to have such difference especially when you know that these two API work TargetList depends on the Front API.
So Front should implement:
Front.watchFronts(type, onFrontAvailable, onFrontDestroyed)Front.unwatchFronts(type, onFrontAvailable, onFrontDestroyed)
Where onFrontAvailable is the equivalent of the callback passed to Front.onFront
and onFrontDestroyed the equivalent of Front.onFrontDestroyed.
|
||
Updated•5 years ago
|
Priority: -- → P3
|
Assignee | |
Updated•5 years ago
|
Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 1•5 years ago
|
||
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fd6c0265761e
Refactor Front.onFront+onFrontDestroyed into Front.watchFronts. r=jdescottes
|
||
Comment 3•5 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox72:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72
You need to log in
before you can comment on or make changes to this bug.
Description
•