Closed Bug 728478 Opened 8 years ago Closed 8 years ago

Consolidate about redirectors

Categories

(SeaMonkey :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.10

People

(Reporter: neil, Assigned: neil)

References

Details

Attachments

(1 file)

We currently have eight (count 'em, eight!) about redirectors and bug 492329 is about to add a ninth... both Firefox and Thunderbird have consolidated all of their redirectors into one file, although Thunderbird at least uses JavaScript.
Attached patch Proposed patchSplinter Review
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #598434 - Flags: review?(iann_bugzilla)
Comment on attachment 598434 [details] [diff] [review]
Proposed patch

>+++ b/suite/common/src/nsAbout.js

>+  lifeFlags: SCRIPT,
should be SCRIPT | HIDE,

>+  sessionrestoreFlags: SCRIPT | HIDE,
should be SCRIPT,

r=me with that fixed.
Attachment #598434 - Flags: review?(iann_bugzilla) → review+
(In reply to Ian Neal from comment #2)
> (From update of attachment 598434 [details] [diff] [review])
> >+  lifeFlags: SCRIPT,
> should be SCRIPT | HIDE,
Oops.

> >+  sessionrestoreFlags: SCRIPT | HIDE,
> should be SCRIPT,
Although it used to, it doesn't actually work standalone any more...
Pushed changeset 6bc285618138 to comm-central.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.10
Depends on: 735567
You need to log in before you can comment on or make changes to this bug.