Implement color management for ImageData

third_party/WebKit/Source/core/html/Float32ImageData.h

 /*
  * 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
  *     notice, this list of conditions and the following disclaimer in the
  *     documentation and/or other materials provided with the distribution.
  * 3.  Neither the name of Apple Computer, Inc. ("Apple") nor the names of
  *     its contributors may be used to endorse or promote products derived
  *     from this software without specific prior written permission.
  *
  * THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
  * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
  * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
  * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
  * 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.
  */
 
-#ifndef ImageData_h
-#define ImageData_h
+#ifndef Float32ImageData_h
+#define Float32ImageData_h
 
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "core/CoreExport.h"
 #include "core/dom/DOMTypedArray.h"
+#include "core/html/ImageData.h"

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

Why is this needed?

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

To have access to ImageDataColorSpace enum and strings.

 #include "core/imagebitmap/ImageBitmapSource.h"
 #include "platform/geometry/IntRect.h"
 #include "platform/geometry/IntSize.h"
 #include "platform/heap/Handle.h"
 #include "wtf/Compiler.h"
+#include "wtf/text/WTFString.h"
 
 namespace blink {
 
 class ExceptionState;
 class ImageBitmapOptions;
 
-class CORE_EXPORT ImageData final : public GarbageCollectedFinalized<ImageData>,
-                                    public ScriptWrappable,
-                                    public ImageBitmapSource {
+class CORE_EXPORT Float32ImageData final
+    : public GarbageCollectedFinalized<Float32ImageData>,
+      public ScriptWrappable,
+      public ImageBitmapSource {
   DEFINE_WRAPPERTYPEINFO();
 
  public:
-  static ImageData* create(const IntSize&);
-  static ImageData* create(const IntSize&, DOMUint8ClampedArray*);
-  static ImageData* create(unsigned width, unsigned height, ExceptionState&);
-  static ImageData* create(DOMUint8ClampedArray*,
-                           unsigned width,
-                           ExceptionState&);
-  static ImageData* create(DOMUint8ClampedArray*,
-                           unsigned width,
-                           unsigned height,
-                           ExceptionState&);
+  static Float32ImageData* create(const IntSize&);
+  static Float32ImageData* create(const IntSize&, DOMFloat32Array*);
+  static Float32ImageData* create(unsigned width,
+                                  unsigned height,
+                                  ExceptionState&);
+  static Float32ImageData* create(unsigned width,
+                                  unsigned height,
+                                  String colorSpace,
+                                  ExceptionState&);
+  static Float32ImageData* create(DOMFloat32Array*,
+                                  unsigned width,
+                                  ExceptionState&);
+  static Float32ImageData* create(DOMFloat32Array*,
+                                  unsigned width,
+                                  unsigned height,
+                                  ExceptionState&);
+  static Float32ImageData* create(DOMFloat32Array*,
+                                  unsigned width,
+                                  unsigned height,
+                                  String colorSpace,
+                                  ExceptionState&);
 
   IntSize size() const { return m_size; }
   int width() const { return m_size.width(); }
   int height() const { return m_size.height(); }
-  const DOMUint8ClampedArray* data() const { return m_data.get(); }
-  DOMUint8ClampedArray* data() { return m_data.get(); }
+  String colorSpace() const { return m_colorSpace; }
+  const DOMFloat32Array* data() const { return m_data.get(); }
+  DOMFloat32Array* data() { return m_data.get(); }
 
   // ImageBitmapSource implementation
   IntSize bitmapSourceSize() const override { return m_size; }
   ScriptPromise createImageBitmap(ScriptState*,
                                   EventTarget&,
                                   Optional<IntRect> cropRect,
                                   const ImageBitmapOptions&,
                                   ExceptionState&) override;
 
   DEFINE_INLINE_TRACE() { visitor->trace(m_data); }
 
   WARN_UNUSED_RESULT v8::Local<v8::Object> associateWithWrapper(
       v8::Isolate*,
       const WrapperTypeInfo*,
       v8::Local<v8::Object> wrapper) override;
 
  private:
-  ImageData(const IntSize&, DOMUint8ClampedArray*);
+  Float32ImageData(const IntSize&,
+                   DOMFloat32Array*,
+                   String = kLegacyImageDataColorSpaceName);
 
-  static bool validateConstructorArguments(DOMUint8ClampedArray*,
+  static bool validateConstructorArguments(DOMFloat32Array*,
                                            unsigned width,
                                            unsigned&,
                                            ExceptionState&);
+  static bool validateConstructorArguments(DOMFloat32Array*,
+                                           unsigned width,
+                                           unsigned&,
+                                           String,
+                                           ExceptionState&);
 
   IntSize m_size;
-  Member<DOMUint8ClampedArray> m_data;
+  String m_colorSpace;

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

You should not store this as a string. Use an enum, and define an attribute
getter that converts to a string.

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

This is an older version. Fixed now.

+  Member<DOMFloat32Array> m_data;
 };
 
 }  // namespace blink
 
-#endif  // ImageData_h
+#endif  // Float32ImageData_h