Bug 1530467 - Add Array<T> support to xpcom rust bindings, r=froydnj

Differential Revision: https://phabricator.services.mozilla.com/D21046

--HG--
extra : moz-landing-system : lando

Author: mystor

Cargo.lock

@@ -423,10 +423,9 @@ def nativeType(self, calltype):
 
     def rustType(self, calltype):
         if self.name == 'nsresult':
-            return "%s::nserror::nsresult" % (calltype != 'in' and '*mut ' or '')
+            return "%s::nserror::nsresult" % ('*mut ' if 'out' in calltype else '')
 
-        return "%s%s" % (calltype != 'in' and '*mut ' or '',
-                         self.name)
+        return "%s%s" % ('*mut ' if 'out' in calltype else '', self.name)
 
     def __str__(self):
         return "typedef %s %s\n" % (self.type, self.name)
@@ -468,6 +467,8 @@ def nativeType(self, calltype):
     def rustType(self, calltype):
         if rustBlacklistedForward(self.name):
             raise RustNoncompat("forward declaration %s is unsupported" % self.name)
+        if calltype == 'element':
+            return 'RefPtr<%s>' % self.name
         return "%s*const %s" % (calltype != 'in' and '*mut ' or '',
                                 self.name)
 
@@ -599,6 +600,11 @@ def rustType(self, calltype, const=False, shared=False):
             prefix += '*mut '
 
         if self.specialtype == 'nsid':
+            if 'element' in calltype:
+                if self.isPtr(calltype):
+                    raise IDLError("Array<nsIDPtr> not yet supported. "
+                                   "File an XPConnect bug if you need it.", self.location)
+                return self.nativename
             return prefix + self.nativename
         if self.specialtype in ['cstring', 'utf8string']:
             if 'element' in calltype:
@@ -762,6 +768,8 @@ def nativeType(self, calltype, const=False):
                              '*' if 'out' in calltype else '')
 
     def rustType(self, calltype, const=False):
+        if calltype == 'element':
+            return 'RefPtr<%s>' % self.name
         return "%s*const %s" % ('*mut ' if 'out' in calltype else '',
                                 self.name)
 
@@ -1331,9 +1339,16 @@ def nativeType(self, calltype):
             return base
 
     def rustType(self, calltype):
-        # NOTE: To add Rust support, ensure 'element' is handled correctly in
-        # all rustType callees.
-        raise RustNoncompat("Array<...> types")
+        if calltype == 'legacyelement':
+            raise IDLError("[array] Array<T> is unsupported", self.location)
+
+        base = 'thin_vec::ThinVec<%s>' % self.type.rustType('element')
+        if 'out' in calltype:
+            return '*mut %s' % base
+        elif 'in' == calltype:
+            return '*const %s' % base
+        else:
+            return base
 
 
 TypeId = namedtuple('TypeId', 'name params')

xpcom/idl-parser/xpidl/xpidl.py

@@ -9,3 +9,4 @@ nsstring = { path = "../nsstring" }
 nserror = { path = "../nserror" }
 threadbound = "0.1"
 xpcom_macros = { path = "xpcom_macros" }
+thin-vec = { version = "0.1.0", features = ["gecko-ffi"] }

xpcom/rust/xpcom/Cargo.toml

@@ -15,6 +15,7 @@ extern crate libc;
 extern crate nsstring;
 extern crate nserror;
 extern crate threadbound;
+extern crate thin_vec;
 
 // re-export the xpcom_macros macro
 #[macro_use]