[bindings] Avoid using custom binding for HTMLOptionsCollection.length attribute

Source/core/html/HTMLOptionsCollection.cpp

Index: Source/core/html/HTMLOptionsCollection.cpp
diff --git a/Source/core/html/HTMLOptionsCollection.cpp b/Source/core/html/HTMLOptionsCollection.cpp
index 1b9c61989d690657f43387ff68b19730de456912..5f6ff463af1840761c0242a330ade96f4edaa094 100644
--- a/Source/core/html/HTMLOptionsCollection.cpp
+++ b/Source/core/html/HTMLOptionsCollection.cpp
@@ -91,6 +91,8 @@ void HTMLOptionsCollection::setSelectedIndex(int index)
 
 void HTMLOptionsCollection::setLength(unsigned length, ExceptionState& exceptionState)
 {
+    if (length > 0x7fffffffu)

[davve, 2015/08/18 12:04:30]

Will this mean we'll get the following behavior?

'.options.length = a' with a in [0,10000] will set length to 'a'.
'.options.length = a' with a in [10001,2147483647] will set length to 10000*.
'.options.length = a' with a in [2147483648,inf) will leave length unchanged.

Would it be worthwhile to align the two latter cases somehow?

*Since we have the maxSelectItems limit in HTMLSelectElement.cpp

[vivekg, 2015/08/18 12:30:32]

Yes makes sense. So the simplified condition would be just:

if (length > maxSelectItems)
   return;

+        return;
     toHTMLSelectElement(ownerNode()).setLength(length, exceptionState);
 }