allHeadersPopup is not used anywhere in mail/mailnews

RESOLVED FIXED

Status

MailNews Core
Backend
--
minor
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

2.52 KB, patch
Karsten Düsterloh
: review+
philor
: review+
neil@parkwaycc.co.uk
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

10 years ago
Created attachment 335159 [details] [diff] [review]
Remove AllHeadersPopup

[[ 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

10 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

10 years ago
Comment on attachment 335159 [details] [diff] [review]
Remove AllHeadersPopup

My, that's old!
Attachment #335159 - Flags: review?(mnyromyr) → review+

Updated

10 years ago
Attachment #335159 - Flags: superreview?(neil) → superreview+
Attachment #335159 - Flags: review?(philringnalda) → review+
(Assignee)

Updated

10 years ago
Keywords: checkin-needed
http://hg.mozilla.org/comm-central/index.cgi/rev/cc43c37b4566
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.