On 05/24/2012 09:14 AM, SULLIVAN, BRYAN L wrote:
Thanks to the inestimable help of the W3C staff I am now plugged into
the mercurial mainline and have uploaded the first stab at the Push
API
> http://dvcs.w3.org/hg/push/raw-file/default/index.html
A couple of notes on the WebIDL:
* PushManager has
|PushService requestRemotePermission ([Optional] DOMString url);|,
should be ... (optional DOMString url), though the spec doesn't
appear to define what happens when url is omitted; same for
checkRemotePermission.
* There does not appear to be a need to make PushManager and PushService
use [NoInterfaceObject], so please remove that.
* PushService.readyState is a DOMString, but the spec defines it to
return the unsigned short constants CONNECTING/OPEN/CLOSED. Please
get rid of the constants.
"The requestUrl attribute must return the absolute URL where the webapp
server can send Push service messages to this webapp." should probably
refer to |serviceUrl|, not |requestUrl|.
A reference to DOM4 for the DOMError interface should probably be added.
Also, the "resolve a url" cross-reference is broken.
HTH
Ms2ger
