Bug 1550545, part 1 - Remove the unused isScriptable methods. r=nika

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

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

xpcom/idl-parser/xpidl/xpidl.py

@@ -16,9 +16,6 @@
 
 """A type conforms to the following pattern:
 
-    def isScriptable(self):
-        'returns True or False'
-
     def nativeType(self, calltype):
         'returns a string representation of the native type
         calltype must be 'in', 'out', 'inout', or 'element'
@@ -124,9 +121,6 @@ def __init__(self, name, nativename, rustname, signed=False, maybeConst=False):
         self.signed = signed
         self.maybeConst = maybeConst
 
-    def isScriptable(self):
-        return True
-
     def isPointer(self):
         """Check if this type is a pointer type - this will control how pointers act"""
         return self.nativename.endswith('*')
@@ -415,9 +409,6 @@ def resolve(self, parent):
         if not isinstance(self.realtype, (Builtin, Native, Typedef)):
             raise IDLError("Unsupported typedef target type", self.location)
 
-    def isScriptable(self):
-        return self.realtype.isScriptable()
-
     def nativeType(self, calltype):
         return "%s %s" % (self.name, '*' if 'out' in calltype else '')
 
@@ -456,9 +447,6 @@ def resolve(self, parent):
 
         parent.setName(self)
 
-    def isScriptable(self):
-        return True
-
     def nativeType(self, calltype):
         if calltype == 'element':
             return 'RefPtr<%s>' % self.name
@@ -524,18 +512,6 @@ def __eq__(self, other):
     def resolve(self, parent):
         parent.setName(self)
 
-    def isScriptable(self):
-        if self.specialtype is None:
-            return False
-
-        if self.specialtype == 'promise':
-            return self.modifier == 'ptr'
-
-        if self.specialtype == 'nsid':
-            return self.modifier is not None
-
-        return self.modifier == 'ref'
-
     def isPtr(self, calltype):
         return self.modifier == 'ptr'
 
@@ -655,9 +631,6 @@ def resolve(self, parent):
 
         parent.setName(self)
 
-    def isScriptable(self):
-        return True  # All DOM objects are script exposed.
-
     def nativeType(self, calltype):
         if calltype == 'element':
             return 'RefPtr<%s>' % self.native
@@ -755,12 +728,6 @@ def resolve(self, parent):
         if self.countEntries() > 250 and not self.attributes.builtinclass:
             raise IDLError("interface '%s' has too many entries" % self.name, self.location)
 
-    def isScriptable(self):
-        # NOTE: this is not whether *this* interface is scriptable... it's
-        # whether, when used as a type, it's scriptable, which is true of all
-        # interfaces.
-        return True
-
     def nativeType(self, calltype, const=False):
         if calltype == 'element':
             return 'RefPtr<%s>' % self.name
@@ -983,9 +950,6 @@ def resolve(self, iface):
     def count(self):
         return 0
 
-    def isScriptable(self):
-        return True
-
     def nativeType(self, calltype):
         if 'out' in calltype:
             return "%s::%s *" % (self.iface.name, self.basename)
@@ -1297,9 +1261,6 @@ def __init__(self, basetype):
         self.type = basetype
         self.location = self.type.location
 
-    def isScriptable(self):
-        return self.type.isScriptable()
-
     def nativeType(self, calltype, const=False):
         if 'element' in calltype:
             raise IDLError("nested [array] unsupported", self.location)
@@ -1333,9 +1294,6 @@ def name(self):
     def resolve(self, idl):
         idl.getName(self.type, self.location)
 
-    def isScriptable(self):
-        return self.type.isScriptable()
-
     def nativeType(self, calltype):
         if calltype == 'legacyelement':
             raise IDLError("[array] Array<T> is unsupported", self.location)