Checking instanceof PixelArray

lib/context2d.js

@@ -483,6 +483,5 @@ Context2d.prototype.createImageData = function(width, height){
     height = width.height;
     width = width.width;
   }
-  var arr = new PixelArray(width, height);
+  return new ImageData(new PixelArray(width, height));
-  return new ImageData(arr);
 };

src/CanvasGradient.cc

@@ -36,8 +36,6 @@ Handle<Value>
 Gradient::New(const Arguments &args) {
   HandleScope scope;
 
-  // TODO: separate concerns / validate args
-
   // Linear
   if (4 == args.Length()) {
     Gradient *grad = new Gradient(

src/ImageData.cc

@@ -36,8 +36,12 @@ ImageData::Initialize(Handle<Object> target) {
 Handle<Value>
 ImageData::New(const Arguments &args) {
   HandleScope scope;
-  // TODO: arg assertions
+  Local<Object> obj = args[0]->ToObject();
-  PixelArray *arr = ObjectWrap::Unwrap<PixelArray>(args[0]->ToObject());
+
+  if (!PixelArray::constructor->HasInstance(obj))
+    return ThrowException(Exception::TypeError(String::New("CanvasPixelArray expected")));
+
+  PixelArray *arr = ObjectWrap::Unwrap<PixelArray>(obj);
   ImageData *imageData = new ImageData(arr);
   args.This()->Set(String::NewSymbol("data"), args[0]);
   imageData->Wrap(args.This());