232597 - In <msgViewPickerOverlay.js>, remove unused |const kPersonalAddressbookURI|

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Serge Gautherie (:sgautherie)]

Spun off from bug 162432 comment 47.

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) <msgViewPickerOverlay.js> [Checked in: Comment 2]

Fixes
{{
Error: redeclaration of const kPersonalAddressbookURI
Source File: chrome://messenger/content/msgViewPickerOverlay.js
Line: 23
--> ABSearchDialog.xul
    -> mailWindowOverlay.xul <- contents.rdf.in <- msgViewPickerOverlay.xul ->
msgViewPickerOverlay.js -> kPersonalAddressbookURI  // removed (useless !!)
    -> abCommon.js -> kPersonalAddressbookURI ! (used)
{
/mailnews/extensions/mailviews/resources/content/msgViewPickerOverlay.xul, line

31 -- <script src="chrome://messenger/content/msgViewPickerOverlay.js"/>

/mailnews/extensions/mailviews/resources/content/contents.rdf.in, line 26 --
<RDF:li>chrome://messenger/content/msgViewPickerOverlay.xul</RDF:li>

Question: <mailWindowOverlay.xul> "used" in many places...
http://landfill.bugzilla.org/mxr-test/seamonkey/search?string=mailWindowOverlay.xul&find=&filter=


Answer: Safe to remove, after cross-checking:
http://landfill.bugzilla.org/mxr-test/seamonkey/search?string=kPersonalAddressbookURI&find=&filter=

http://landfill.bugzilla.org/mxr-test/seamonkey/search?string=abCommon.js&find=&filter=

}
}}
When opening "Tools > Search Adresses...".

Comment 2

[[e-mail]]

Fix checked in.