Implement color management for ImageData

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

Index: third_party/WebKit/Source/core/html/ImageData.idl
diff --git a/third_party/WebKit/Source/core/html/ImageData.idl b/third_party/WebKit/Source/core/html/ImageData.idl
index d52d253a70049da153f86574ce21973a3bae0ad3..3b2e7bc11e3c04c106115ff4bc5fa1bfec313ea3 100644
--- a/third_party/WebKit/Source/core/html/ImageData.idl
+++ b/third_party/WebKit/Source/core/html/ImageData.idl
@@ -28,14 +28,16 @@
 
 // https://html.spec.whatwg.org/#dom-imagedata
 
+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"),

[Justin Novosad, 2016/12/08 20:58:59]

We need to hide this behind a flag.  To do this, instead of using 'optional' you
should create an overload of the constructor that takes a mandatory colorspace
argument, and hide that behind the runtime flag.

[zakerinasab, 2016/12/08 22:13:52]

Done.

+    Constructor(Uint8ClampedArray data, unsigned long sw, optional unsigned long sh, optional ImageDataColorSpace colorSpace),

[Justin Novosad, 2016/12/08 20:58:59]

same here.

[zakerinasab, 2016/12/08 22:13:52]

Done.

     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;
+    readonly attribute ImageDataColorSpace colorSpace;
 };