Closed
Bug 696598
Opened 13 years ago
Closed 13 years ago
Add default value for extensions.autoDisableScopes
Categories
(SeaMonkey :: Preferences, defect)
SeaMonkey
Preferences
Tracking
(Not tracked)
VERIFIED
FIXED
seamonkey2.7
People
(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)
References
Details
Attachments
(2 files)
4.34 KB,
patch
|
iannbugzilla
:
review+
|
Details | Diff | Splinter Review |
1.67 KB,
patch
|
philip.chee
:
review+
|
Details | Diff | Splinter Review |
Changeset 15e15fead382: Bug 693743: Some 3rd party add-ons are getting installed into the profile and are not disabled on start-up // Disable add-ons installed into the shared user and shared system areas by // default. This does not include the application directory. See the SCOPE // constants in AddonManager.jsm for values to use here pref("extensions.autoDisableScopes", 15); The scope constants are here: http://mxr.mozilla.org/comm-central/source/mozilla/toolkit/mozapps/extensions/AddonManager.jsm#1258 Pref first introduced through: Changeset eee41544cb84: Bug 476430: Disable third-party add-ons by default and offer them to the user from which we're also missing the nsBrowserGlue.js part (which we might want to add here, too).
Assignee | ||
Comment 1•13 years ago
|
||
Porting patch, nsSuiteGlue.js part yet untested.
Comment 2•13 years ago
|
||
Does anybody know how this affects extensions delivered via {application}/distribution/extensions ?
Comment 3•13 years ago
|
||
(In reply to Philip Chee from comment #2) > Does anybody know how this affects extensions delivered via > {application}/distribution/extensions ? I /think/ the bit of weight 4 (as in 15 but not in 10, the two values used in various Fx builds) excludes them. I hope I'm wrong but… I guess the guy to ask this from might be Mossop.
Comment 4•13 years ago
|
||
It doesn't affect them at all
Comment on attachment 568907 [details] [diff] [review] patch [Checkin: comment 9] I presume with this patch that we now pass the relevant tests from toolkit/mozapps/extensions/test/xpcshell that were introduced by bug 693743 ?
Attachment #568907 -
Flags: review?(iann_bugzilla) → review+
Assignee | ||
Comment 6•13 years ago
|
||
(In reply to Ian Neal from comment #5) > I presume with this patch that we now pass the relevant tests from > toolkit/mozapps/extensions/test/xpcshell that were introduced by bug 693743 ? No. test_startup.js still fails through its dependency head_addons.js which fails with: TEST-UNEXPECTED-FAIL | /path/to/my/mozilla/seamonkey-central/mozilla/_tests/xpcshell/toolkit/mozapps/extensions/test/xpcshell/head_addons.js | [] == ["modern@themes.mozilla.org"] - See following stack: JS frame :: /path/to/my/comm-central/mozilla/testing/xpcshell/head.js :: do_throw :: line 453 JS frame :: /path/to/my/comm-central/mozilla/testing/xpcshell/head.js :: _do_check_eq :: line 547 JS frame :: /path/to/my/comm-central/mozilla/testing/xpcshell/head.js :: do_check_eq :: line 568 JS frame :: /path/to/my/mozilla/seamonkey-central/mozilla/_tests/xpcshell/toolkit/mozapps/extensions/test/xpcshell/head_addons.js :: check_startup_changes :: line 454 which is probably because the test only excludes the default theme (which happens to have the same ID for both FF and SM) in check_startup_changes on line 450 but not Modern. I guess that can only be fixed in the test itself, but I'm not sure whether adding yet another static value is exactly the right way to go there... Probably easiest, though (if accepted). Dave, what would be my chances there if I filed a bug?
Comment 7•13 years ago
|
||
(In reply to Jens Hatlak (:InvisibleSmiley) from comment #6) > (In reply to Ian Neal from comment #5) > > I presume with this patch that we now pass the relevant tests from > > toolkit/mozapps/extensions/test/xpcshell that were introduced by bug 693743 ? > > No. test_startup.js still fails through its dependency head_addons.js which > fails with: > > TEST-UNEXPECTED-FAIL | > /path/to/my/mozilla/seamonkey-central/mozilla/_tests/xpcshell/toolkit/ > mozapps/extensions/test/xpcshell/head_addons.js | [] == > ["modern@themes.mozilla.org"] - See following stack: > JS frame :: /path/to/my/comm-central/mozilla/testing/xpcshell/head.js :: > do_throw :: line 453 > JS frame :: /path/to/my/comm-central/mozilla/testing/xpcshell/head.js :: > _do_check_eq :: line 547 > JS frame :: /path/to/my/comm-central/mozilla/testing/xpcshell/head.js :: > do_check_eq :: line 568 > JS frame :: > /path/to/my/mozilla/seamonkey-central/mozilla/_tests/xpcshell/toolkit/ > mozapps/extensions/test/xpcshell/head_addons.js :: check_startup_changes :: > line 454 > > which is probably because the test only excludes the default theme (which > happens to have the same ID for both FF and SM) in check_startup_changes on > line 450 but not Modern. I guess that can only be fixed in the test itself, > but I'm not sure whether adding yet another static value is exactly the > right way to go there... Probably easiest, though (if accepted). Dave, what > would be my chances there if I filed a bug? I think the right thing there is to filter out any add-ons with IDs that don't end with "@tests.mozilla.org".
Assignee | ||
Comment 8•13 years ago
|
||
(In reply to Dave Townsend (:Mossop) from comment #7) > I think the right thing there is to filter out any add-ons with IDs that > don't end with "@tests.mozilla.org". Filed bug 698524.
Assignee | ||
Comment 9•13 years ago
|
||
Comment on attachment 568907 [details] [diff] [review] patch [Checkin: comment 9] http://hg.mozilla.org/comm-central/rev/58837246ee0b
Attachment #568907 -
Attachment description: patch → patch [Checkin: comment 9]
Assignee | ||
Updated•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.7
Comment 10•13 years ago
|
||
> + var win = this.getMostRecentBrowserWindow(); You forgot to define getMostRecentBrowserWindow() anywhere in nsSuiteGlue.js Introduced in: http://hg.mozilla.org/mozilla-central/rev/a131999fa900 Modified in: http://hg.mozilla.org/mozilla-central/rev/36a5be172c91 http://hg.mozilla.org/mozilla-central/rev/3d137f5214ad
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 11•13 years ago
|
||
> _onBrowserStartup: function(aWindow) And besides, you already have a window. > { > if (Services.prefs.getBoolPref("plugins.update.notifyUser")) > this._showPluginUpdatePage(aWindow); > > + // For any add-ons that were installed disabled and can be enabled offer > + // them to the user. > + var win = this.getMostRecentBrowserWindow(); > + var browser = win.gBrowser; Use getBrowser() like notifyBox below (our gBrowser isn't a lazy getter). > + var changedIDs = AddonManager.getStartupChanges(AddonManager.STARTUP_CHANGE_INSTALLED); > + AddonManager.getAddonsByIDs(changedIDs, function(aAddons) { > + aAddons.forEach(function(aAddon) { > + // If the add-on isn't user disabled or can't be enabled then skip it. > + if (!aAddon.userDisabled || !(aAddon.permissions & AddonManager.PERM_CAN_ENABLE)) > + return; > + > + browser.selectedTab = browser.addTab("about:newaddon?id=" + aAddon.id); > + }) > + }); > + > var notifyBox = aWindow.getBrowser().getNotificationBox(); reuse |var browser| ?
Assignee | ||
Comment 12•13 years ago
|
||
Fix "what Ratty said". Thanks for the heads-up!
Attachment #572309 -
Flags: review?(philip.chee)
Comment 13•13 years ago
|
||
Comment on attachment 572309 [details] [diff] [review] browser fix [Checkin: comment 14] OK. Tested by dropping an extension into seamonkey/extensions/ r=me p.s. please file a Themes->Modern bug to style the about:newaddon UI in modern.
Attachment #572309 -
Flags: review?(philip.chee) → review+
Assignee | ||
Comment 14•13 years ago
|
||
Comment on attachment 572309 [details] [diff] [review] browser fix [Checkin: comment 14] http://hg.mozilla.org/comm-central/rev/bd56a0704795
Attachment #572309 -
Attachment description: patch → browser fix [Checkin: comment 14]
Assignee | ||
Updated•13 years ago
|
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Comment 15•13 years ago
|
||
The preference exists (default=15) in this build: Mozilla/5.0 (X11; Linux x86_64; rv:10.0a1) Gecko/20111107 Firefox/10.0a1 SeaMonkey/2.7a1 ID:20111107193531 (Mozilla tinderbox-build) It wasn't listed in about:config in: Mozilla/5.0 (X11; Linux x86_64; rv:10.0a1) Gecko/20111106 Firefox/10.0a1 SeaMonkey/2.7a1 ID:20111106024605 (own-built build, timestamp corrected for timezone difference)
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•