Closed Bug 1981455 Opened 4 months ago Closed 4 months ago

Split BrowserGlue's listeners/observers forwarding pattern up into a dedicated module

Categories

(Firefox :: General, task, P1)

Desktop
All
task

Tracking

()

RESOLVED FIXED
143 Branch
Tracking Status
firefox143 --- fixed

People

(Reporter: Gijs, Assigned: Gijs)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

Where callers / notifiers of the observer topic are JS based, using the category manager is probably a better fit, but there's other observer-based code for which that doesn't work and so making this a tiny bit more generic and using it for some of the observers that currently live in browser.js would help with loading code just-in-time and make it easier to refactor that code.

The Bugbug bot thinks this bug should belong to the 'Core::JavaScript Engine' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: General → JavaScript Engine
Product: Firefox → Core

(In reply to BugBot [:suhaib / :marco/ :calixte] from comment #1)

The Bugbug bot thinks this bug should belong to the 'Core::JavaScript Engine' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

☹️

Component: JavaScript Engine → General
Product: Core → Firefox
Depends on: 1981463
Blocks: 1880918
Pushed by gijskruitbosch@gmail.com: https://github.com/mozilla-firefox/firefox/commit/12a86641d725 https://hg.mozilla.org/integration/autoland/rev/7dbc34ab048e factor observer forwarding code out from BrowserGlue into a dedicated module for reuse, r=firefox-desktop-core-reviewers ,dao
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 143 Branch
QA Whiteboard: [qa-triage-done-c144/b143]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: