Re: [w3ctag/spec-reviews] Custom Paint API to CR (#140)

* property paintWorklet is on the Window interface. Is there a reason it's global? Could there be an alternative home for this?
* Not sure the "Note: This is how the class should look" should use WebIDL to describe the expected parameter. I think it would be much clearer to just use a JavaScript example (a class)
* registerPaint(name, paintCtor):step 2. "NotSupportedError" seems logically wrong. The closest match I found in the WebIDL spec was https://heycam.github.io/webidl/#inuseattributeerror, but that didn't seem quite right either. Is it time for a new "UniqueValueError" or some such (this can't be the first time this case is needed)

Editorial Nits:
* "be the result of lookuping up" (looking up?)
* No need to define VoidFunction. It's defined for everyone in https://heycam.github.io/webidl/#VoidFunction
 


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3ctag/spec-reviews/issues/140#issuecomment-256214435