548937 - Refactor fake services so it's easier to add more

Status: RESOLVED FIXED

(Firefox :: Sync, defect)

Description

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

Fennec doesn't have some browser specific services, so we need to fake them. Private browsing is already there, but we'll need to add more (e.g. SessionStore).

So to prevent Utils.lazySvc from becoming bloated by putting the services there, they should be elsewhere.

Comment 1

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

Attachment: Patch v0.1

I put fakeServices in Utils, but it can be moved.

Comment 2

[Ed Lee :Mardak]

Comment on attachment 429239 [details] [diff] [review]
Patch v0.1

>+++ b/source/modules/util.js
>       if (!Cc[cid]) {
>+        if (cid in fakeServices)
>+          svc = fakeServices[cid];
Eh.. I would just do svc = FakeSvc[cid]
>         if (svc == null)
>           log.warn("Component " + cid + " doesn't exist on this platform.");
And svc == null would check okay. But then it'll trigger a strict warning :p
I suppose the if check is okay.

>+  fakeServices: {
Just make this a top level FakeSvc object, but don't export it.

Comment 3

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

(In reply to comment #2)
> (From update of attachment 429239 [details] [diff] [review])
> >+++ b/source/modules/util.js
> >       if (!Cc[cid]) {
> >+        if (cid in fakeServices)
> >+          svc = fakeServices[cid];
> Eh.. I would just do svc = FakeSvc[cid]

I can do that.

> >         if (svc == null)
> >           log.warn("Component " + cid + " doesn't exist on this platform.");
> And svc == null would check okay. But then it'll trigger a strict warning :p
> I suppose the if check is okay.

So turns out that strict warnings won't show up during assignment. So I think we could do without any issues.
> svc = FakeSvc[cid];
> if (!svc)

Comment 4

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

Pushed http://hg.mozilla.org/labs/weave/rev/87f9860d8e5a

Comment 5

[Ed Lee :Mardak]

http://hg.mozilla.org/labs/weave/rev/9e4817ec4791
Revert svc/svc == null change from 87f9860d8e5a.