Implement ImageData constructors.

Source/core/html/ImageData.cpp

 /*
  * Copyright (C) 2008 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 11 matching lines @@
  * (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.
  */
 
 #include "config.h"
 #include "core/html/ImageData.h"
 
+#include "bindings/v8/ExceptionState.h"
+#include "core/dom/ExceptionCode.h"
+
 namespace WebCore {
 
 PassRefPtr<ImageData> ImageData::create(const IntSize& size)
 {
     Checked<int, RecordOverflow> dataSize = 4;
     dataSize *= size.width();
     dataSize *= size.height();
     if (dataSize.hasOverflowed())
         return nullptr;
 
     return adoptRef(new ImageData(size));
 }
 
 PassRefPtr<ImageData> ImageData::create(const IntSize& size, PassRefPtr<Uint8ClampedArray> byteArray)
 {
     Checked<int, RecordOverflow> dataSize = 4;
     dataSize *= size.width();
     dataSize *= size.height();
     if (dataSize.hasOverflowed())
         return nullptr;
 
     if (dataSize.unsafeGet() < 0
         || static_cast<unsigned>(dataSize.unsafeGet()) > byteArray->length())
         return nullptr;
 
     return adoptRef(new ImageData(size, byteArray));
 }
 
+PassRefPtr<ImageData> ImageData::create(unsigned width, unsigned height, ExceptionState& exceptionState)
+{
+    if (!width || !height) {
+        exceptionState.throwDOMException(IndexSizeError, String::format("The source %s is zero or non-finite.", width ? "height" : "width"));

[Justin Novosad, 2014/03/19 19:35:54]

how can an unsigned be non-finite?

[sof, 2014/03/19 19:41:10]

ToUInt32() maps Inf and NaNs (including non-numbers mapped to such by
ToNumber()) to 0. "not a number" is a better term to use here.

+        return nullptr;
+    }
+
+    Checked<unsigned, RecordOverflow> dataSize = 4;
+    dataSize *= width;
+    dataSize *= height;
+    if (dataSize.hasOverflowed()) {
+        exceptionState.throwDOMException(IndexSizeError, "The requested image size exceeds the supported range.");
+        return nullptr;
+    }
+
+    RefPtr<ImageData> imageData = adoptRef(new ImageData(IntSize(width, height)));
+    imageData->data()->zeroFill();
+    return imageData;

[Inactive, 2014/03/19 19:07:14]

nit: .release()

+}
+
+PassRefPtr<ImageData> ImageData::create(Uint8ClampedArray* data, unsigned width, unsigned height, ExceptionState& exceptionState)
+{
+    if (!data) {
+        exceptionState.throwTypeError("Expected a Uint8ClampedArray as first argument.");
+        return nullptr;
+    }
+    if (!width) {
+        exceptionState.throwDOMException(IndexSizeError, "The source width is zero or non-finite.");
+        return nullptr;
+    }
+
+    Uint8ClampedArray* clampedData = static_cast<Uint8ClampedArray*>(data);

[Inactive, 2014/03/19 19:07:14]

Why is this cast needed?

I actually don't understand by we need this local variable at all. We should be
able to use "data" directly unless I am missing something.

[sof, 2014/03/19 22:19:16]

Yes, doesn't make much sense; gone. [I belatedly discovered that the IDL
compiler supported Uint8ClampedArray instance checking, and not just
ArrayBufferViews; this is a remnant.]

+    unsigned clampedLength = clampedData->length() / 4;

[Justin Novosad, 2014/03/19 19:35:54]

According to the spec you need to throw an InvalidStateError if
clampedData->length() is not a multiple of 4. You should perform that check
before this division.

[sof, 2014/03/19 22:19:16]

Thanks, quite right. Addressed + expanded the testing in this area.

+    if (!clampedLength) {
+        exceptionState.throwDOMException(IndexSizeError, "The input data has a zero byte length.");
+        return nullptr;
+    }
+    if (clampedLength % width) {
+        exceptionState.throwDOMException(IndexSizeError, "The byte length of the input data is not a multiple of (4 * width).");
+        return nullptr;
+    }
+    if (!height) {
+        height = clampedLength / width;
+    } else if (height != clampedLength / width) {
+        exceptionState.throwDOMException(IndexSizeError, "The byte length of the input data is not equal to (4 * width * height).");
+        return nullptr;
+    }
+
+    return adoptRef(new ImageData(IntSize(width, height), clampedData));
+}
+
 ImageData::ImageData(const IntSize& size)
     : m_size(size)
     , m_data(Uint8ClampedArray::createUninitialized(size.width() * size.height() * 4))
 {
     ScriptWrappable::init(this);
 }
 
 ImageData::ImageData(const IntSize& size, PassRefPtr<Uint8ClampedArray> byteArray)
     : m_size(size)
     , m_data(byteArray)
 {
     ASSERT_WITH_SECURITY_IMPLICATION(static_cast<unsigned>(size.width() * size.height() * 4) <= m_data->length());
     ScriptWrappable::init(this);
 }
 
 }