1527772 - Decouple folder creation from RDF

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Ben Campbell]

Folders are currently constructed by looking up the correct nsIFactory and calling createInstance().

The factories all have the form"@mozilla.org/rdf/resource-factory;1?name=" + scheme, which kind of implies they're registered as part of the RDF system.
They need to be freed from any RDF entanglements.

Comment 1

[Ben Campbell]

My patch(es) for Bug 1527764 will move the folder creation into the folder-lookup-service, so easiest to wait until that stuff lands before worrying about this one.

Comment 2

[Ben Campbell]

Attachment: decouple-folder-creation-from-rdf-1.patch

This patch builds on top of the two RDF-removal patches in Bug 1527764.
It just shuffles about some naming to get folder creation out of the RDF code.

Comment 3

[:aceman]

Comment on attachment 9055367 [details] [diff] [review]
decouple-folder-creation-from-rdf-1.patch

Review of attachment 9055367 [details] [diff] [review]:
-----------------------------------------------------------------

Nice, thanks.

::: mailnews/base/src/folderLookupService.js
@@ +66,5 @@
>  
>      // Check that uri has an active scheme, in case this folder is from
>      // an extension that is currently disabled or hasn't started up yet.
>      let scheme = uri.match(/\w*/)[0];
> +    let contractID = "@mozilla.org/mail/folder-factory;1?name=" + scheme;

Declare it const while there.

::: mailnews/base/util/nsMsgDBFolder.h
@@ -8,5 @@
>  
>  #include "mozilla/Attributes.h"
>  #include "msgCore.h"
>  #include "nsIMsgFolder.h"
> -#include "nsRDFResource.h"

Great, last include ;)

Comment 4

[Jorg K (GMT+1)]

So this is waiting for bug 1527764 which is waiting for bug 1534163 which is ready now, right? So how about rebasing everything as necessary and doing a final try run?

Comment 5

[Ben Campbell]

Attachment: decouple-folder-creation-from-rdf-2.patch

Rebased.
To apply on top of the two patches in Bug 1527764.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/3970c1407633
Move folder-factory lookup out of RDF. r=aceman