Closed Bug 451823 Opened 13 years ago Closed 13 years ago

allHeadersPopup is not used anywhere in mail/mailnews

Categories

(MailNews Core :: Backend, defect)

Product:
MailNews Core
Component:
Backend
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: philip.chee, Assigned: philip.chee)

Details

Attachments

(1 file)

Remove AllHeadersPopup
2.52 KB, patch
mnyromyr
: review+
philor
: review+
neil
: superreview+
Details | Diff | Splinter Review 
[[ there really should be a "shared" or "common" component in "MailNews Core" ]]

<http://mxr.mozilla.org/seamonkey/search?string=AllHeadersPopup>

This popup is defined but not used anywhere. This patch removes the cruft.
Attachment #335159 - Flags: superreview?(neil)
Attachment #335159 - Flags: review?(mnyromyr)
Comment on attachment 335159 [details] [diff] [review]
Remove AllHeadersPopup

philor: please review the /mail/ part of the diff. Thanks.
Attachment #335159 - Flags: review?(philringnalda)
Comment on attachment 335159 [details] [diff] [review]
Remove AllHeadersPopup

My, that's old!
Attachment #335159 - Flags: review?(mnyromyr) → review+
Attachment #335159 - Flags: superreview?(neil) → superreview+
Attachment #335159 - Flags: review?(philringnalda) → review+
Keywords: checkin-needed
http://hg.mozilla.org/comm-central/index.cgi/rev/cc43c37b4566
Status: NEW → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.