Improve TypeError messages from failed array conversions.

Source/bindings/v8/custom/V8MessageEventCustom.cpp

Index: Source/bindings/v8/custom/V8MessageEventCustom.cpp
diff --git a/Source/bindings/v8/custom/V8MessageEventCustom.cpp b/Source/bindings/v8/custom/V8MessageEventCustom.cpp
index 21c4f0e438a703721f63c368238681c3cd44a71c..a44393f521ba873426a41b1e9146dccc4ac09a5f 100644
--- a/Source/bindings/v8/custom/V8MessageEventCustom.cpp
+++ b/Source/bindings/v8/custom/V8MessageEventCustom.cpp
@@ -118,7 +118,7 @@ void V8MessageEvent::initMessageEventMethodCustom(const v8::FunctionCallbackInfo
 
     if (!isUndefinedOrNull(args[7])) {
         portArray = adoptPtr(new MessagePortArray);
-        if (!getMessagePortArray(args[7], *portArray, args.GetIsolate()))
+        if (!getMessagePortArray(args[7], 8, *portArray, args.GetIsolate()))

[Mike West, 2013/10/24 07:08:54]

This isn't really your fault, since this call is already full of magic numbers,
but I'd appreciate it if you changed "7" and "8" in this line to something
readable. 'const int portArrayIndex = 7' and 'portArrayIndex + 1' or something
similar.

[sof, 2013/10/24 21:16:41]

good idea, done.

I filed http://code.google.com/p/chromium/issues/detail?id=311332 on
deprecating&removing this method.

             return;
     }
     event->initMessageEvent(typeArg, canBubbleArg, cancelableArg, originArg, lastEventIdArg, sourceArg, portArray.release());