Closed Bug 631773 Opened 13 years ago Closed 13 years ago

Clean up the global scope a bit [CreateMailWindowGlobals()]

Categories

(SeaMonkey :: MailNews: Message Display, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b3

People

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

Details

Attachments

(1 file, 1 obsolete file)

An itsy bitsy clean up bug focusing on CreateMailWindowGlobals()

Q.v.
Bug 416666 (part 2) Clean up Thunderbird's global scope a bit.
Bug 440616 (part 1 v2) Clean up Thunderbird's global scope a bit more (mail session and 3pane commands).
Attached patch Patch v1.0 WIP (obsolete) — Splinter Review
>    //Create datasources
> +  var prefix = "@mozilla.org/rdf/datasource;1?name=";
> +  var accountManagerDSContractID = prefix + "msgaccountmanager";
> +  var folderDSContractID         = prefix + "mailnewsfolders";
> +
>    accountManagerDataSource = Components.classes[accountManagerDSContractID].getService();
>    folderDataSource         = Components.classes[folderDSContractID].getService();

If I do .getService(nsiRDFDataSource) here. Can I remove the QIs from the rest of suite/mailnews/ ?
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #510006 - Flags: review?(mnyromyr)
Attachment #510006 - Flags: feedback?(neil)
(In reply to comment #1)
> >    accountManagerDataSource = Components.classes[accountManagerDSContractID].getService();
> >    folderDataSource         = Components.classes[folderDSContractID].getService();
> 
> If I do .getService(nsiRDFDataSource) here. Can I remove the QIs from the rest
> of suite/mailnews/ ?

If you mean the four QIs to nsIRDFDataSource, then yes, they could go.
Attachment #510006 - Flags: feedback?(neil) → feedback+
> neil@parkwaycc.co.uk      2011-02-05 07:09:28 PST
> 
> (In reply to comment #1)
>>>    accountManagerDataSource = Components.classes[accountManagerDSContractID].getService();
>>>    folderDataSource         = Components.classes[folderDSContractID].getService();
>> 
>> If I do .getService(nsiRDFDataSource) here. Can I remove the QIs from the rest
>> of suite/mailnews/ ?
> 
> If you mean the four QIs to nsIRDFDataSource, then yes, they could go.
Jolly Good.
Attachment #510006 - Attachment is obsolete: true
Attachment #510268 - Flags: superreview?(neil)
Attachment #510268 - Flags: review?(mnyromyr)
Attachment #510006 - Flags: review?(mnyromyr)
Attachment #510268 - Flags: superreview?(neil) → superreview+
Attachment #510268 - Flags: review?(mnyromyr) → review+
Pushed to comm-central
http://hg.mozilla.org/comm-central/rev/dbc2f57e7b63
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: