Closed
Bug 451823
Opened 14 years ago
Closed 14 years ago
allHeadersPopup is not used anywhere in mail/mailnews
Categories
(MailNews Core :: Backend, defect)
MailNews Core
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philip.chee, Assigned: philip.chee)
Details
Attachments
(1 file)
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)
![]() |
Assignee | |
Comment 1•14 years ago
|
||
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 2•14 years ago
|
||
Comment on attachment 335159 [details] [diff] [review] Remove AllHeadersPopup My, that's old!
Attachment #335159 -
Flags: review?(mnyromyr) → review+
Updated•14 years ago
|
Attachment #335159 -
Flags: superreview?(neil) → superreview+
Updated•14 years ago
|
Attachment #335159 -
Flags: review?(philringnalda) → review+
![]() |
Assignee | |
Updated•14 years ago
|
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•