Bug 1407432 - Remove unused builtins arg from Sandbox; r=froydnj draft
authorMike Shal <mshal@mozilla.com>
Wed, 08 Nov 2017 19:58:28 -0500
changeset 695827 3148d9f0ad58008092a46daa909f27c522fa278c
parent 695826 2182084e3c43c2bb3590d1b8cea6a9ad1ed42087
child 695828 553f94811a274d037ebcc82346a35fb16fd4d2c3
push id88557
push userbmo:mshal@mozilla.com
push dateThu, 09 Nov 2017 22:17:50 +0000
reviewersfroydnj
bugs1407432
milestone58.0a1
Bug 1407432 - Remove unused builtins arg from Sandbox; r=froydnj MozReview-Commit-ID: JVukxIg04AT
python/mozbuild/mozbuild/frontend/sandbox.py
--- a/python/mozbuild/mozbuild/frontend/sandbox.py
+++ b/python/mozbuild/mozbuild/frontend/sandbox.py
@@ -108,20 +108,20 @@ class Sandbox(dict):
         'None': None,
         'False': False,
         'True': True,
         'sorted': alphabetical_sorted,
         'int': int,
         'set': set,
     })
 
-    def __init__(self, context, builtins=None, finder=default_finder):
+    def __init__(self, context, finder=default_finder):
         """Initialize a Sandbox ready for execution.
         """
-        self._builtins = builtins or self.BUILTINS
+        self._builtins = self.BUILTINS
         dict.__setitem__(self, '__builtins__', self._builtins)
 
         assert isinstance(self._builtins, ReadOnlyDict)
         assert isinstance(context, Context)
 
         # Contexts are modeled as a stack because multiple context managers
         # may be active.
         self._active_contexts = [context]