File:  [mozdev] / annozilla / service / TODO
Revision 1.2: download - view: text, annotated - select for diffs - revision graph
Wed Mar 5 18:17:03 2003 UTC (14 years, 7 months ago) by rainking
Branches: MAIN
- merged from ANNOTATIONS_SERVICE branch

* fix serverName to be serverURI in IDL for annotation, and then fix everything that effects

* make serverURI readonly

* change everything that we can from throwing an exception to returning null instead, which seems to work

* change things called URL to URI when necessary

* add code to check status of XMLHttpRequest objects, and make error messages better in general in the remote server error callback functions

* test autoLoad on same page in a row, to ensure that annotations don't come through twice

* in AnnotationsService.onEndLoad, testing the autoLoadServersResponded == serverCount could be a problem if servers are added or removed.  test this case.


* surround code that makes client callbacks and then does more stuff in try/catch, with a console-print of any error message in the catch.  this prevents client code from jacking the service
 - update --> XPCOM may do this for us, so unneccessary

* design data structure setup such that each app registered for autoLoad gets one AnnotationsEnum with all of the annotations for its servers
 - this should be done by having an array for each application, and as servers respond in we need a server --> array of application map
 - this could be done each time we autoLoad, but that's expensive.  instead, we could update this as observers and servers and autoLoad stuff changes for each app, and keep this sort of thing in the app itself.  since autoLoad happens much more often than updating preferences, this is the way to go.

* Need to clone annotations *and* enumerator *every* time we give out annotations to multiple observers--this should be the job of the ObserverAlerter for multiple observers within an application, as well as the AnnotationsService whenever a particular annotation is sent to multiple applications

FreeBSD-CVSweb <>