Closed
Bug 1148475
Opened 9 years ago
Closed 6 years ago
Kill nsIBrowserGlue
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
People
(Reporter: mak, Assigned: mak)
References
Details
in bug 1094888 I'm going to remove the Places smart bookmarks API exposed through it. getMostRecentBrowserWindow() is not really useful since we have RecentWindow.jsm, so we could make it issue deprecation warnings even immediately. The only left thing is sanitize(window). It is used by preferences, but I suppose they could just instantiate a Sanitizer and use that... And it used by <command id="Tools:Sanitize" oncommand="Cc['@mozilla.org/browser/browserglue;1'].getService(Ci.nsIBrowserGlue).sanitize(window);"/> for which I suppose we could find an alternative in browser.js Having one less XPCOM crossing and interface around would be nice.
Flags: firefox-backlog+
Comment 1•9 years ago
|
||
Kill it with fire. Marco, could you mentor this?
Points: --- → 3
Flags: qe-verify-
Flags: needinfo?(mak77)
Assignee | ||
Comment 2•9 years ago
|
||
I hope so, let me start by filing dependencies.
Mentor: mak77
Flags: needinfo?(mak77)
Comment 3•8 years ago
|
||
Hi Marco, I'd like to work on this bug. Can you guide me through?
Assignee | ||
Comment 4•8 years ago
|
||
I'm sorry. this is not currently actionable cause it has dependency bugs unfixed.
Assignee | ||
Updated•8 years ago
|
Mentor: mak77
Assignee | ||
Comment 5•6 years ago
|
||
Fixed by bug 1167237!
Assignee: nobody → mak77
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•