Closed Bug 1561880 Opened 5 years ago Closed 5 years ago

Refactor browser window listeners to use EveryWindow.jsm

Categories

(Firefox :: Messaging System, task, P3)

task

Tracking

()

RESOLVED FIXED
Firefox 71
Iteration:
71.1 - Sept 2 - 15
Tracking Status
firefox71 --- fixed

People

(Reporter: andreio, Assigned: andreio)

References

(Blocks 1 open bug)

Details

(Keywords: github-merged)

Attachments

(1 file)

In ASRouterTriggerListeners.jsm there are a lot of operations that involve attaching event listeners on windows that would benefit from using EveryWindow.jsm

Additional refactoring for checkStartupFinished (same file): there is a delayedStartupPromise on every window object that we could use instead of our function. I suspect we might not even need it since EveryWindow waits for it before attaching the listeners.

Priority: -- → P3
Blocks: 1581195
Status: NEW → RESOLVED
Iteration: --- → 71.1 - Sept 2 - 15
Closed: 5 years ago
Keywords: github-merged
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71
Assignee: nobody → andrei.br92
See Also: → 1660201
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: