HTMLOptionsCollection remove() argument should be long, not unsigned long

LayoutTests/fast/js/resources/select-options-remove.js

 description("This test checks the behavior of the remove() method on the select.options object.");
 
 var select1 = document.getElementById("select1");
 var value;
 
 debug("1.1 Remove (object) from empty Options");
 value = document.createElement("DIV");
 shouldBe("select1.options.remove(value)", "undefined");
 shouldBe("select1.options.length", "0");
 shouldBe("select1.selectedIndex", "-1");
@@ skipping 222 matching lines @@
 shouldBe("select2.options.length", "3");
 shouldBe("select2.selectedIndex", "1");
 shouldBe("select2.options[1].value", "'O'");
 debug("");
 
 debug("2.17 Detach select element");
 shouldNotBe("select2.parentNode", "null");
 shouldBe("select2.remove()", "undefined");
 shouldBeNull("select2.parentNode");
 debug("");
+
+debug("2.18 Remove index -2147483649 from non-empty Options");
+shouldBe("select2.options.remove(-2147483649)", "undefined");

[philipj_slow, 2015/06/29 09:33:05]

Checking the return value of remove() doesn't seem meaningful, as it's always
void.

Is it possible to write a test that would fail with the old code and pass with
the new? It seems to me that there isn't, because we can only test with a small
number of options (certainly less than two billion) and the difference is only
observable around INT_MAX and INT_MIN. Can you confirm?

[shiva.jm, 2015/06/30 05:33:23]

Done.

+debug("");
+
+debug("2.19 Remove index -2147483648 from non-empty Options");
+shouldBe("select2.options.remove(-2147483648)", "undefined");
+debug("");
+
+debug("2.20 Remove index -2147483647 from non-empty Options");
+shouldBe("select2.options.remove(-2147483647)", "undefined");
+debug("");
+
+debug("2.21 Remove index 2147483649 from non-empty Options");
+shouldBe("select2.options.remove(2147483649)", "undefined");
+debug("");
+
+debug("2.22 Remove index 2147483648 from non-empty Options");
+shouldBe("select2.options.remove(2147483648)", "undefined");
+debug("");
+
+debug("2.23 Remove index 2147483647 from non-empty Options");
+shouldBe("select2.options.remove(2147483647)", "undefined");
+debug("");