Re: [whatwg] proposal: Add support for masking to canvas

On 1/30/13 12:37 AM, Rik Cabanier wrote:
>     enum CanvasMaskType { luminosity, alpha};

Last nits.  Need quotes around the enum values.  So:

   enum CanvasMaskType { "luminosity", "alpha" };

>     void maskImage(MaskOptions options);

I believe that has to be:

   void maskImage(optional MaskOptions options);

to be valid WebIDL.

Apart from that, the IDL parts look fine.  Can't speak to the actual 
graphics bits.  ;)

-Boris

Received on Wednesday, 30 January 2013 05:49:42 UTC