Implement color management for ImageData

third_party/WebKit/Source/core/html/ImageData.idl

 /*
  * Copyright (C) 2008, 2009 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 10 matching lines @@
  * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
  * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
  * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
  * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 // https://html.spec.whatwg.org/#dom-imagedata
 
+//[
+//    Constructor(unsigned long sw, unsigned long sh),

[Justin Novosad, 2016/12/08 18:16:35]

Remove this comment block

[zakerinasab, 2016/12/08 20:40:52]

Done.

+//    Constructor(Uint8ClampedArray data, unsigned long sw, optional unsigned long sh),
+//    Exposed=(Window,Worker),
+//    RaisesException=Constructor,
+//] interface ImageData {
+//    readonly attribute unsigned long width;
+//    readonly attribute unsigned long height;
+//    readonly attribute Uint8ClampedArray data;
+//};
+
+enum ImageDataColorSpace { "legacy-srgb", "srgb", "linear-rgb" };
+
 [
-    Constructor(unsigned long sw, unsigned long sh),
-    Constructor(Uint8ClampedArray data, unsigned long sw, optional unsigned long sh),
+    Constructor(unsigned long sw, unsigned long sh, optional ImageDataColorSpace colorSpace = "legacy-srgb"),
+    Constructor(Uint8ClampedArray data, unsigned long sw, optional unsigned long sh, optional ImageDataColorSpace colorSpace),
     Exposed=(Window,Worker),
     RaisesException=Constructor,
 ] interface ImageData {
     readonly attribute unsigned long width;
     readonly attribute unsigned long height;
-    // TODO(foolip): Expose data.
-    // readonly attribute Uint8ClampedArray data;
+    readonly attribute Uint8ClampedArray data;

[Justin Novosad, 2016/12/08 18:16:35]

This should not be read-only

[zakerinasab, 2016/12/08 20:40:52]

Resolved. Remains readonly.

+    readonly attribute ImageDataColorSpace colorSpace;
 };