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

[Boris Zbarsky]

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