Open Bug 446292 Opened 16 years ago Updated 7 years ago

In <messengerdnd.js>, overlapping (= potentially "conflicting") |var RDF|

Categories

(SeaMonkey :: General, defect)

defect
Not set
minor

Tracking

(Not tracked)

People

(Reporter: sgautherie, Unassigned, NeedInfo)

References

(Blocks 1 open bug, )

Details

Steps:
1. Open Browser.
2. Open (Ctrl+2) MailNews.

I tried to s/var/const/ the RDF definition, and got
{{
Error: redeclaration of var RDF
Source File: chrome://messenger/content/messengerdnd.js
}}

As a test, I moved <messengerdnd.js> to be first in the JS list in <mailWindow.js>, and got
{{
Error: redeclaration of const RDF
Source File: chrome://messenger/content/mailWindow.js

Error: redeclaration of const RDF
Source File: chrome://communicator/content/bookmarks/bookmarks.js
}}

These 3 files eventually define the "same" |var RDF| value.
Even if it is working (fine) as it is,
I just hit a case where this becomes an issue :-/

This seems to be the same kind of problem as bug 446027 (-> bug 446281)...
What is the proper way to solve this (case) ?
Rename the constant/variable(s) ? Share a common definition ? ...
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.0.2pre) Gecko/2008071502 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

This is with SM;

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.0.2pre) Gecko/2008071303 Thunderbird/3.0a2pre] (nightly) (W2Ksp4)

but TB has the same issue.
(Without the bookmarks file, obviously.)
Product: Core → MailNews Core
Target Milestone: mozilla1.9.1a1 → ---
The messengednd.js file seems to only exist in Seamonkey. But there does not seem to be any declaration of RDF these days.
Component: Backend → General
Flags: needinfo?(philip.chee)
Product: MailNews Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.