Closed Bug 1148475 Opened 9 years ago Closed 6 years ago

Kill nsIBrowserGlue

Categories

(Firefox :: General, defect)

defect
Not set
normal
Points:
3

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+
Kill it with fire. Marco, could you mentor this?
Points: --- → 3
Flags: qe-verify-
Flags: needinfo?(mak77)
I hope so, let me start by filing dependencies.
Mentor: mak77
Flags: needinfo?(mak77)
Depends on: 1167224
Depends on: 1167229
Depends on: 1167237
Hi Marco, I'd like to work on this bug. Can you guide me through?
I'm sorry. this is not currently actionable cause it has dependency bugs unfixed.
Mentor: mak77
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.