[EnforceRange] doesn't enforce range of a short

Source/bindings/scripts/code_generator_v8.pm

 # Copyright (C) 2005, 2006 Nikolas Zimmermann <zimmermann@kde.org>
 # Copyright (C) 2006 Anders Carlsson <andersca@mac.com>
 # Copyright (C) 2006 Samuel Weinig <sam.weinig@gmail.com>
 # Copyright (C) 2006 Alexey Proskuryakov <ap@webkit.org>
 # Copyright (C) 2006 Apple Computer, Inc.
 # Copyright (C) 2007, 2008, 2009, 2012 Google Inc.
 # Copyright (C) 2009 Cameron McCormack <cam@mcc.id.au>
 # Copyright (C) Research In Motion Limited 2010. All rights reserved.
 # Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
 # Copyright (C) 2012 Ericsson AB. All rights reserved.
@@ skipping 5351 matching lines @@
     my $getIsolate = shift;
 
     my $intConversion = $extendedAttributes->{"EnforceRange"} ? "EnforceRange" : "NormalConversion";
 
     return "$value->BooleanValue()" if $type eq "boolean";
     return "static_cast<$type>($value->NumberValue())" if $type eq "float" or $type eq "double";
 
     if ($intConversion ne "NormalConversion") {
         return "toInt8($value, $intConversion, ok)" if $type eq "byte";
         return "toUInt8($value, $intConversion, ok)" if $type eq "octet";
-        return "toInt32($value, $intConversion, ok)" if $type eq "long" or $type eq "short";

[eseidel, 2013/11/04 07:14:13]

I see.  We just were mistakenly treating it 32bit.

Thanks for fixing.

-        return "toUInt32($value, $intConversion, ok)" if $type eq "unsigned long" or $type eq "unsigned short";
+        return "toInt16($value, $intConversion, ok)" if $type eq "short";
+        return "toUInt16($value, $intConversion, ok)" if $type eq "unsigned short";
+        return "toInt32($value, $intConversion, ok)" if $type eq "long";
+        return "toUInt32($value, $intConversion, ok)" if $type eq "unsigned long";
         return "toInt64($value, $intConversion, ok)" if $type eq "long long";
         return "toUInt64($value, $intConversion, ok)" if $type eq "unsigned long long";
     } else {
         return "toInt8($value)" if $type eq "byte";
         return "toUInt8($value)" if $type eq "octet";
-        return "toInt32($value)" if $type eq "long" or $type eq "short";
-        return "toUInt32($value)" if $type eq "unsigned long" or $type eq "unsigned short";
+        return "toInt16($value)" if $type eq "short";
+        return "toUInt16($value)" if $type eq "unsigned short";
+        return "toInt32($value)" if $type eq "long";
+        return "toUInt32($value)" if $type eq "unsigned long";
         return "toInt64($value)" if $type eq "long long";
         return "toUInt64($value)" if $type eq "unsigned long long";
     }
     return "static_cast<Range::CompareHow>($value->Int32Value())" if $type eq "CompareHow";
     return "toWebCoreDate($value)" if $type eq "Date";
 
     if ($type eq "DOMString" or IsEnumType($type)) {
         return $value;
     }
 
@@ skipping 894 matching lines @@
 
     return 1 if $interface->extendedAttributes->{"CustomToV8"};
     return 1 if $interface->extendedAttributes->{"SpecialWrapFor"};
     return 1 if InheritsInterface($interface, "Document");
     return 1 if SVGTypeNeedsToHoldContextElement($interface->name);
 
     return 0;
 }
 
 1;