Closed Bug 1590401 Opened 11 months ago Closed 10 months ago

Refactor Front.onFront/offFront/onFrontDestroyed/offFrontDestroyed into Front.watchFronts/unwatchFronts

Categories

(DevTools :: Framework, enhancement, P3)

enhancement

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.
Priority: -- → P3
Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fd6c0265761e
Refactor Front.onFront+onFrontDestroyed into Front.watchFronts. r=jdescottes
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72
You need to log in before you can comment on or make changes to this bug.