The default bug view has changed. See this FAQ.

Delete toolkit/obsolete

RESOLVED FIXED in Firefox 52

Status

()

Toolkit
General
RESOLVED FIXED
5 months ago
14 days ago

People

(Reporter: erahm, Assigned: erahm)

Tracking

({addon-compat})

unspecified
mozilla52
addon-compat
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 months ago
This is an obsolete interface used by addons. Per discussion on IRC we'd like to just go ahead and delete this.

As a bonus there's some nsISupportsArray usage in the directory, so we'd be getting rid of usage of that as well.

Comment 1

5 months ago
These interfaces have been deprecated for a very long time, so it should be safe to remove them at this point. A lot of add-ons still use a few of them, particularly nsUserSettings.js and strres.js, but fortunately most of them are not compatible with recent (or any) versions of Firefox.

The rest can easily be updated to use better alternatives like Preferences.jsm and and <stringbundle> nodes.
(Assignee)

Comment 2

5 months ago
Created attachment 8803579 [details] [diff] [review]
Delete toolkit/obsolete

This removes 'toolkit/obsolete'. I'm not sure the best reviewer here, Kris feel free to redirect.

MozReview-Commit-ID: JZfi0W2wqCL
Attachment #8803579 - Flags: review?(kmaglione+bmo)
(Assignee)

Updated

5 months ago
Assignee: nobody → erahm
Status: NEW → ASSIGNED

Comment 3

5 months ago
Comment on attachment 8803579 [details] [diff] [review]
Delete toolkit/obsolete

Review of attachment 8803579 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the lingering in-tree references fixed.

::: toolkit/obsolete/content/inlineSpellCheckUI.js
@@ -3,5 @@
> - * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
> -
> -Components.utils.import("resource://gre/modules/InlineSpellChecker.jsm");
> -
> -var InlineSpellCheckerUI = new InlineSpellChecker();

This needs to be moved to browser.js, and then this file needs to be removed from global-scripts.inc, web-panels.xul, and import-browserjs-globals.js

::: toolkit/obsolete/jar.mn
@@ -3,5 @@
> -# file, You can obtain one at http://mozilla.org/MPL/2.0/.
> -
> -toolkit.jar:
> -   content/global/globalOverlay.xul             (content/globalOverlay.xul)
> -*  content/global/dialogOverlay.xul             (content/dialogOverlay.xul)

It looks like dialogOverlay.xul is still used 3 places. cookieAcceptDialog.xul is dead code at this point, so it probably doesn't matter. printPreviewProgress.xul and printProgress.xul should just need the overlay removed and the top-level <window> changed to <dialog>.
Attachment #8803579 - Flags: review?(kmaglione+bmo) → review+
(Assignee)

Comment 4

5 months ago
Thanks for the quick review!

(In reply to Kris Maglione [:kmag] from comment #3)
> Comment on attachment 8803579 [details] [diff] [review]
> Delete toolkit/obsolete
> 
> Review of attachment 8803579 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> r=me with the lingering in-tree references fixed.
> 
> ::: toolkit/obsolete/content/inlineSpellCheckUI.js
> @@ -3,5 @@
> > - * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
> > -
> > -Components.utils.import("resource://gre/modules/InlineSpellChecker.jsm");
> > -
> > -var InlineSpellCheckerUI = new InlineSpellChecker();
> 
> This needs to be moved to browser.js, and then this file needs to be removed
> from global-scripts.inc, web-panels.xul, and import-browserjs-globals.js

I'm assuming just the browser.js under browser/base/content, right?

> ::: toolkit/obsolete/jar.mn
> @@ -3,5 @@
> > -# file, You can obtain one at http://mozilla.org/MPL/2.0/.
> > -
> > -toolkit.jar:
> > -   content/global/globalOverlay.xul             (content/globalOverlay.xul)
> > -*  content/global/dialogOverlay.xul             (content/dialogOverlay.xul)
> 
> It looks like dialogOverlay.xul is still used 3 places.
> cookieAcceptDialog.xul is dead code at this point, so it probably doesn't
> matter. printPreviewProgress.xul and printProgress.xul should just need the
> overlay removed and the top-level <window> changed to <dialog>.

Will do.
(Assignee)

Comment 5

5 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=0edaad91bc80
(Assignee)

Comment 6

5 months ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/2dca0ba58df6abd5c5f5b260644b3b56f6e7e69b
Bug 1312143 - Delete toolkit/obsolete. r=John-Galt

Comment 7

5 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/2dca0ba58df6
Status: ASSIGNED → RESOLVED
Last Resolved: 5 months ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52

Updated

5 months ago
Blocks: 1313906
I wonder if we have data on how many add-ons still use nsPreferences (from chrome://global/content/nsUserSettings.js).
I noticed an old (badly written) add-on misbehaving, and figured out it was because of nsPreferences being undefined.

Comment 9

5 months ago
There are a fair number on AMO. It's hard to say exactly how many, though, since most of them don't support remotely recent versions of Firefox, and many of them don't support Firefox at all.

Either way, these scripts have been deprecated for a very long time. It's time for add-ons that are still being maintained to stop using them.
Depends on: 1255149

Updated

16 days ago
Duplicate of this bug: 916235

Updated

14 days ago
Depends on: 1345860
You need to log in before you can comment on or make changes to this bug.