Implement color management for ImageData

third_party/WebKit/Source/core/imagebitmap/ImageBitmapFactories.cpp

 /*
  * Copyright (c) 2013, Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 19 matching lines @@
 
 #include "core/imagebitmap/ImageBitmapFactories.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/dom/DOMException.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/fileapi/Blob.h"
 #include "core/frame/ImageBitmap.h"
 #include "core/frame/LocalDOMWindow.h"
 #include "core/frame/UseCounter.h"
+#include "core/html/Float32ImageData.h"
 #include "core/html/HTMLCanvasElement.h"
 #include "core/html/HTMLImageElement.h"
 #include "core/html/HTMLVideoElement.h"
 #include "core/html/ImageData.h"
 #include "core/imagebitmap/ImageBitmapOptions.h"
 #include "core/svg/graphics/SVGImage.h"
 #include "core/workers/WorkerGlobalScope.h"
 #include "platform/CrossThreadFunctional.h"
 #include "platform/SharedBuffer.h"
 #include "platform/image-decoders/ImageDecoder.h"
@@ skipping 34 matching lines @@
     return imageElement;
   }
   if (value.isHTMLVideoElement())
     return value.getAsHTMLVideoElement();
   if (value.isHTMLCanvasElement())
     return value.getAsHTMLCanvasElement();
   if (value.isBlob())
     return value.getAsBlob();
   if (value.isImageData())
     return value.getAsImageData();
+  if (RuntimeEnabledFeatures::trueColorRenderingEnabled() ||
+      RuntimeEnabledFeatures::colorCorrectRenderingEnabled()) {
+    if (value.isFloat32ImageData())

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

Would be useful to give helpful guidance to devs trying out this feature.  If a
Float32ImageData was passed, but neither trueColor or colorCorrect are enabled,
we should throw an exception with a message that tells the dev what to do to
make things work.

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

Done.

+      return value.getAsFloat32ImageData();
+  }
   if (value.isImageBitmap())
     return value.getAsImageBitmap();
   ASSERT_NOT_REACHED();
   return nullptr;
 }
 
 ScriptPromise ImageBitmapFactories::createImageBitmapFromBlob(
     ScriptState* scriptState,
     EventTarget& eventTarget,
     ImageBitmapSource* bitmapSource,
@@ skipping 214 matching lines @@
   }
   m_factory->didFinishLoading(this);
 }
 
 DEFINE_TRACE(ImageBitmapFactories::ImageBitmapLoader) {
   visitor->trace(m_factory);
   visitor->trace(m_resolver);
 }
 
 }  // namespace blink