532175 - Send event when Weave is ready to register new engines

Status: RESOLVED FIXED

(Cloud Services :: General, enhancement, P3)

Description

[Axel Nennker]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.9.1.5) Gecko/20091102 Firefox/3.5.5
Build Identifier: 

The developer instructions on how to write and integrate a new engine give the advice to modify the original weave code. This is not practical for real world extensions that want to sync their data.

The other way to do it is to use:
Components.utils.import("resource://weave/service.js");
Weave.Engines.register(MyEngine); // MyEngine is your engine class

But it is not guaranteed that EngineManagerSvc already is instanciated.

So it is safe to create an event when the Engine Manager is ready to register engines. Extensions wanting to register engines can then wait for that event.

This is a small change. That should not stop the 1.0 version release.

Reproducible: Always

Comment 1

[Ed Lee :Mardak]

http://hg.mozilla.org/labs/weave/rev/e106915cbb70
Notify with "weave:service:ready" so observers can lazily load and wait to add engines, Weave UI.