Closed Bug 1993788 Opened 2 months ago Closed 1 month ago

Restart with add-ons disabled is broken in SeaMonkey 2.53.22b1

Categories

(SeaMonkey :: General, defect)

SeaMonkey 2.53
defect

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
seamonkey2.53
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: frg, Assigned: frg)

References

(Regression)

Details

(Keywords: regression, Whiteboard: SM2.53.22)

Attachments

(1 file)

The code still uses smile and the changes to suite/smile/src/smileApplication.js in bug 1614671 broke the restart. The ChromeUtils api has some different behaviour here and we need to put XPCOMUtils in again for now. We should still pick parts of Bug 1388561 for 2.53 too to remove the general smile usage from our code. This is only kept for old extensions.

Timestamp: 11.10.2025, 13:30:08
Error: TypeError: Application.restart is not a function
Source File: chrome://communicator/content/utilityOverlay.js
Line: 686

Summary: Restart wint add-ons disabled is broken in Seaonkey 2.53.22b1 → Restart with add-ons disabled is broken in Seaonkey 2.53.22b1

[Approval Request Comment]
Regression caused by (bug #): bug 1614671
User impact if declined: restart does not work.
Testing completed (on m-c, etc.): 2.53.23b1 pre
Risk to taking this patch (and alternatives if risky): trivial
String changes made by this patch: --

Attachment #9519578 - Flags: review?(iannbugzilla)
Attachment #9519578 - Flags: approval-comm-release?

Do we need some spin off bugs to remove the dependency of "restart with add-ons disable" on smile?
and/or fixing ChromeUtils behaviour?

Flags: needinfo?(frgrahl)

Comment on attachment 9519578 [details] [diff] [review]
1993788-25322.patch

[Triage Comment]
As a workaround LGTM r/a=me

Attachment #9519578 - Flags: review?(iannbugzilla)
Attachment #9519578 - Flags: review+
Attachment #9519578 - Flags: approval-comm-release?
Attachment #9519578 - Flags: approval-comm-release+
Summary: Restart with add-ons disabled is broken in Seaonkey 2.53.22b1 → Restart with add-ons disabled is broken in SeaMonkey 2.53.22b1

Bug 1388561 removed all Smile usage in SeaMonkey and is now in to 2.53 also. What is left is add-on usage but the Smile code should probably removed and any classic add-ons still maintained fixed. I doubt there are any usable left anyway. Places code in Smile eg. probably needs a full replacement and imho not worth it. I would add bug 1435621 and maybe bug 1354205 to 2.53 and get over it the hard way.
I don't think we will remove XPCOMUtils from 2.53 so can use this one as a workaround. So far I have not seen other broken stuff but the add-ons sdk might be affected too.

Flags: needinfo?(frgrahl)
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.53
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: