Properly handle variadic arguments preceded by default values

I broke that in #8197.

components/script/dom/bindings/codegen/CodegenRust.py

@@ -1189,22 +1189,23 @@ class CGArgumentConverter(CGThing):
                 template, variadicConversion, declType, "slot")]
 
             arg = "arg%d" % index
-            vec = "Vec::with_capacity((%(argc)s - %(index)s) as usize)" % {'argc': argc, 'index': index}
 
             if argument.type.isGeckoInterface():
-                code = "let mut %s = RootedVec::new();\n*%s = %s;\n" % (arg, arg, vec)
+                vec = "RootedVec::new()"
                 innerConverter.append(CGGeneric("%s.push(JS::from_ref(&*slot));" % arg))
             else:
-                code = "let mut %s = %s;\n" % (arg, vec)
+                vec = "vec![]"
                 innerConverter.append(CGGeneric("%s.push(slot);" % arg))
-            inner = CGIndenter(CGList(innerConverter, "\n"), 4).define()
+            inner = CGIndenter(CGList(innerConverter, "\n"), 8).define()
 
-            code += """\
-for variadicArg in %(index)s..%(argc)s {
+            self.converter = CGGeneric("""\
+let mut %(arg)s = %(vec)s;
+if %(argc)s > %(index)s {
+    %(arg)s.reserve(%(argc)s as usize - %(index)s);
+    for variadicArg in %(index)s..%(argc)s {
 %(inner)s
-}""" % {'argc': argc, 'index': index, 'inner': inner}
-
-            self.converter = CGGeneric(code)
+    }
+}""" % {'arg': arg, 'argc': argc, 'index': index, 'inner': inner, 'vec': vec})
 
     def define(self):
         return self.converter.define()

components/script/dom/testbinding.rs

@@ -362,6 +362,7 @@ impl TestBindingMethods for TestBinding {
     // fn PassOptionalNullableEnumWithNonNullDefault(self, _: Option<TestEnum>) {}
 
     fn PassVariadicBoolean(&self, _: Vec<bool>) {}
+    fn PassVariadicBooleanAndDefault(&self, _: bool, _: Vec<bool>) {}
     fn PassVariadicByte(&self, _: Vec<i8>) {}
     fn PassVariadicOctet(&self, _: Vec<u8>) {}
     fn PassVariadicShort(&self, _: Vec<i16>) {}

components/script/dom/webidls/TestBinding.webidl

@@ -325,6 +325,7 @@ interface TestBinding {
   // void passOptionalNullableUnion2WithNonNullDefault(optional (Event or DOMString)? data = "foo");
 
   void passVariadicBoolean(boolean... args);
+  void passVariadicBooleanAndDefault(optional boolean arg = true, boolean... args);
   void passVariadicByte(byte... args);
   void passVariadicOctet(octet... args);
   void passVariadicShort(short... args);

tests/wpt/mozilla/tests/mozilla/variadic-interface.html

@@ -8,4 +8,9 @@ test(function() {
     var t = new TestBinding;
     t.passVariadicInterface(new Blob, new Blob);
 }, "Variadic interface arguments work.");
+
+test(function() {
+    var t = new TestBinding;
+    t.passVariadicBooleanAndDefault();
+}, "Default values and variadic arguments work together.");
 </script>