Open Bug 1545408 Opened 6 years ago Updated 7 months ago

Merge EveryWindow.sys.mjs into BrowserWindowTracker.sys.mjs

Categories

(Firefox :: General, task, P3)

task

Tracking

()

Tracking Status
firefox68 --- affected

People

(Reporter: nhnt11, Unassigned)

References

(Blocks 1 open bug)

Details

(In reply to Dão Gottwald [::dao] from bug 1416163 comment #13)

Are you aware of BrowserWindowTracker.jsm that has been added after this bug had been dropped over a year ago? Seems to me that we should extend that module if needed rather than adding another similar one.

Yeah, I'm aware of BrowserWindowTracker. I thought about extending that module but it didn't feel like I could do that without some refactoring. I'll file a bug to investigate merging the two modules. Thanks!

Priority: -- → P5
Priority: P5 → P3
Summary: Investigate merging EveryWindow.jsm into BrowserWindowTracker.jsm → Merge EveryWindow.jsm into BrowserWindowTracker.jsm
Severity: normal → N/A
Component: General → New Tab Page
Summary: Merge EveryWindow.jsm into BrowserWindowTracker.jsm → Merge EveryWindow.sys.mjs into BrowserWindowTracker.sys.mjs
Component: New Tab Page → General
See Also: → 1893847
Blocks: 1893847
See Also: 1893847
You need to log in before you can comment on or make changes to this bug.