Closed Bug 826771 Opened 9 years ago Closed 9 years ago
Manager .xul is broken because Services isn't defined in the scope .
Before Philip's suggested patch, I couldn't remove an (feed) app I've added to the list of apps in the helper apps pane - hitting OK just wouldn't do anything at all (forced to use the canel button). Now I can remove an app from the list, but if I close the prefwindow and open it again, the app reappears in the list.
For more details regarding my issues, see https://bugzilla.mozilla.org/show_bug.cgi?id=824272#c3 (most of them except no 2 are probably unrelated to this bug)
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #702283 - Flags: review?(iann_bugzilla)
Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/507de3e54b36
Comment on attachment 702283 [details] [diff] [review] Patch v1.0 import Services.jsm Bug 667662 and Bug 667674 landed on SeaMonkey 2.4 eek! [Approval Request Comment] Regression caused by (bug #): Bug 667662 and Bug 667674. User impact if declined: broken applications pref-pane. Testing completed (on m-c, etc.): tested on comm-central. Risk to taking this patch (and alternatives if risky): low risk, bustage fix. String changes made by this patch: None.
Comment on attachment 702283 [details] [diff] [review] Patch v1.0 import Services.jsm a=me, I'll leave the rel eng team to make a decision on comm-release
Attachment #702283 - Flags: approval-comm-release? → approval-comm-release+
Pushed to comm-release: http://hg.mozilla.org/releases/comm-release/rev/2ba7869026e0
You need to log in before you can comment on or make changes to this bug.